ensure Other variants are only used by JSON targets

Author: RalfJung

compiler/rustc_target/src/spec/mod.rs

@@ -3218,6 +3218,30 @@ impl Target {
             );
         }
 
+        // Ensure built-in targets don't use the `Other` variants.
+        if kind == TargetKind::Builtin {
+            check!(
+                !matches!(self.arch, Arch::Other(_)),
+                "`Other` variants are only meant for JSON targets"
+            );
+            check!(
+                !matches!(self.os, Os::Other(_)),
+                "`Other` variants are only meant for JSON targets"
+            );
+            check!(
+                !matches!(self.env, Env::Other(_)),
+                "`Other` variants are only meant for JSON targets"
+            );
+            check!(
+                !matches!(self.cfg_abi, CfgAbi::Other(_)),
+                "`Other` variants are only meant for JSON targets"
+            );
+            check!(
+                !matches!(self.llvm_abiname, LlvmAbi::Other(_)),
+                "`Other` variants are only meant for JSON targets"
+            );
+        }
+
         // Check ABI flag consistency, for the architectures where we have proper ABI treatment.
         // To ensure targets are trated consistently, please consult with the team before allowing
         // new cases.