target declaration refactor

Author: Firestar99

crates/rustc_codegen_spirv-types/src/target.rs

@@ -1,6 +1,8 @@
 use std::fmt::{Debug, Formatter};
 use thiserror::Error;
 
+pub const SPIRV_VENDOR: &str = "unknown";
+pub const SPIRV_ARCH: &str = "spirv";
 pub const SPIRV_TARGET_PREFIX: &str = "spirv-unknown-";
 
 /// A well-formed rust-gpu target.

crates/rustc_codegen_spirv/src/builder_spirv.rs

@@ -5,7 +5,7 @@ use crate::builder;
 use crate::codegen_cx::CodegenCx;
 use crate::spirv_type::SpirvType;
 use crate::symbols::Symbols;
-use crate::target::SpirvTarget;
+use crate::target::{SpirvTarget, SpirvVersion};
 use crate::target_feature::TargetFeature;
 use rspirv::dr::{Builder, Instruction, Module, Operand};
 use rspirv::spirv::{
@@ -454,8 +454,8 @@ impl<'tcx> BuilderSpirv<'tcx> {
         target: &SpirvTarget,
         features: &[TargetFeature],
     ) -> Self {
-        let version = target.spirv_version();
-        let memory_model = target.memory_model();
+        let version = target.spirv_version().unwrap();
+        let memory_model = target.memory_model().unwrap();
 
         let mut builder = Builder::new();
         builder.set_version(version.0, version.1);
@@ -492,7 +492,7 @@ impl<'tcx> BuilderSpirv<'tcx> {
 
         add_cap(&mut builder, &mut enabled_capabilities, Capability::Shader);
         if memory_model == MemoryModel::Vulkan {
-            if version < (1, 5) {
+            if version < SpirvVersion::V1_5 {
                 add_ext(&mut builder, sym.spv_khr_vulkan_memory_model);
             }
             add_cap(

crates/rustc_codegen_spirv/src/codegen_cx/mod.rs

@@ -10,11 +10,11 @@ use crate::builder_spirv::{
 use crate::custom_decorations::{CustomDecoration, SrcLocDecoration, ZombieDecoration};
 use crate::spirv_type::{SpirvType, SpirvTypePrinter, TypeCache};
 use crate::symbols::Symbols;
-use crate::target::SpirvTarget;
 
 // HACK(eddyb) avoids rewriting all of the imports (see `lib.rs` and `build.rs`).
 use crate::maybe_pqp_cg_ssa as rustc_codegen_ssa;
 
+use crate::target::SpirvTarget;
 use itertools::Itertools as _;
 use rspirv::dr::{Module, Operand};
 use rspirv::spirv::{Decoration, LinkageType, Word};
@@ -99,7 +99,7 @@ impl<'tcx> CodegenCx<'tcx> {
     pub fn new(tcx: TyCtxt<'tcx>, codegen_unit: &'tcx CodegenUnit<'tcx>) -> Self {
         // Validate the target spec, as the backend doesn't control `--target`.
         let target_tuple = tcx.sess.opts.target_triple.tuple();
-        let target: SpirvTarget = target_tuple.parse().unwrap_or_else(|_| {
+        let target = SpirvTarget::parse(target_tuple).unwrap_or_else(|_| {
             let qualifier = if !target_tuple.starts_with("spirv-") {
                 "non-SPIR-V "
             } else {

crates/rustc_codegen_spirv/src/lib.rs

@@ -136,7 +136,7 @@ mod linker;
 mod spirv_type;
 mod spirv_type_constraints;
 mod symbols;
-mod target;
+pub mod target;
 mod target_feature;
 
 use builder::Builder;

crates/rustc_codegen_spirv/src/linker/test.rs

@@ -13,6 +13,7 @@ use std::sync::{Arc, Mutex};
 // We need to construct an emitter as yet another workaround,
 // see https://github.com/rust-lang/rust/pull/102992.
 extern crate termcolor;
+use crate::target::SpirvTarget;
 use termcolor::{ColorSpec, WriteColor};
 
 // https://github.com/colin-kiegel/rust-pretty-assertions/issues/24
@@ -130,10 +131,7 @@ fn link_with_linker_opts(
                 .unwrap();
         let sopts = rustc_session::config::build_session_options(&mut early_dcx, &matches);
 
-        let target = "spirv-unknown-spv1.0"
-            .parse::<crate::target::SpirvTarget>()
-            .unwrap()
-            .rustc_target();
+        let target = SpirvTarget::UNIVERSAL_1_0.rustc_target();
         let sm_inputs = rustc_span::source_map::SourceMapInputs {
             file_loader: Box::new(rustc_span::source_map::RealFileLoader),
             path_mapping: sopts.file_path_mapping(),