compiletest: avoid invoking rustc with absolute paths.

Author: eddyb

crates/rustc_codegen_spirv/src/codegen_cx/mod.rs

@@ -767,25 +767,6 @@ impl CodegenArgs {
 
         if self.disassemble_globals {
             for inst in module.global_inst_iter() {
-                // HACK: On Windows, paths are printed like `OpString "D:\\dir\\blah"`.
-                // Unfortunately, compiletest will only normalize `D:\dir\blah` to `$DIR/blah` -
-                // one backslash, not two. So, when disassembling for compiletest, check if the
-                // argument to OpString can be parsed as an absolute path, and if it is, replace it
-                // with just the filename component of the path.
-                if inst.class.opcode == Op::String {
-                    let path = Path::new(inst.operands[0].unwrap_literal_string());
-                    if path.is_absolute()
-                        && let Some(file_name) = path.file_name()
-                    {
-                        let mut inst = inst.clone();
-                        inst.operands[0] = Operand::LiteralString(format!(
-                            "$OPSTRING_FILENAME/{}",
-                            file_name.to_string_lossy(),
-                        ));
-                        eprintln!("{}", inst.disassemble());
-                        continue;
-                    }
-                }
                 eprintln!("{}", inst.disassemble());
             }
         }

tests/compiletests/src/main.rs

@@ -65,6 +65,13 @@ fn main() {
     let deps_target_dir = original_target_dir.join("compiletest-deps");
     let compiletest_build_dir = original_target_dir.join("compiletest-results");
 
+    // HACK(eddyb) force `compiletest` to pass `ui/...` relative paths to `rustc`,
+    // which should always end up being the same regardless of the path that the
+    // Rust-GPU repo is checked out at (among other things, this avoids harcoded
+    // `compiletest` limits being hit by e.g. users with slightly longer paths).
+    std::env::set_current_dir(tests_dir).unwrap();
+    let tests_dir = PathBuf::from("");
+
     // Pull in rustc_codegen_spirv as a dynamic library in the same way
     // spirv-builder does.
     let codegen_backend_path = find_rustc_codegen_spirv();

tests/compiletests/ui/dis/asm_op_decorate.stderr

@@ -4,7 +4,7 @@ OpExtension "SPV_EXT_descriptor_indexing"
 OpMemoryModel Logical Simple
 OpEntryPoint Fragment %1 "main"
 OpExecutionMode %1 OriginUpperLeft
-%2 = OpString "$OPSTRING_FILENAME/asm_op_decorate.rs"
+%2 = OpString "$DIR/asm_op_decorate.rs"
 OpName %3 "asm_op_decorate::main"
 OpName %4 "asm_op_decorate::add_decorate"
 OpDecorate %5 Binding 0

tests/compiletests/ui/dis/const-float-cast-optimized.stderr

@@ -3,7 +3,7 @@ OpCapability Float64
 OpMemoryModel Logical Simple
 OpEntryPoint Fragment %1 "main" %2
 OpExecutionMode %1 OriginUpperLeft
-%3 = OpString "$OPSTRING_FILENAME/const-float-cast-optimized.rs"
+%3 = OpString "$DIR/const-float-cast-optimized.rs"
 OpName %2 "output"
 OpDecorate %2 Location 0
 %4 = OpTypeFloat 64

tests/compiletests/ui/dis/const-float-cast.stderr

@@ -3,7 +3,7 @@ OpCapability Float64
 OpMemoryModel Logical Simple
 OpEntryPoint Fragment %1 "main" %2
 OpExecutionMode %1 OriginUpperLeft
-%3 = OpString "$OPSTRING_FILENAME/const-float-cast.rs"
+%3 = OpString "$DIR/const-float-cast.rs"
 OpName %2 "output"
 OpDecorate %2 Location 0
 %4 = OpTypeFloat 32

tests/compiletests/ui/dis/const-from-cast.stderr

@@ -2,7 +2,7 @@ OpCapability Shader
 OpMemoryModel Logical Simple
 OpEntryPoint Fragment %1 "main" %2
 OpExecutionMode %1 OriginUpperLeft
-%3 = OpString "$OPSTRING_FILENAME/const-from-cast.rs"
+%3 = OpString "$DIR/const-from-cast.rs"
 OpName %2 "output"
 OpDecorate %2 Location 0
 %4 = OpTypeInt 32 0

tests/compiletests/ui/dis/const-int-cast.stderr

@@ -2,7 +2,7 @@ OpCapability Shader
 OpMemoryModel Logical Simple
 OpEntryPoint Fragment %1 "main" %2
 OpExecutionMode %1 OriginUpperLeft
-%3 = OpString "$OPSTRING_FILENAME/const-int-cast.rs"
+%3 = OpString "$DIR/const-int-cast.rs"
 OpName %2 "output"
 OpDecorate %2 Location 0
 %4 = OpTypeInt 32 0

tests/compiletests/ui/dis/const-narrowing-cast.stderr

@@ -3,7 +3,7 @@ OpCapability Int8
 OpMemoryModel Logical Simple
 OpEntryPoint Fragment %1 "main" %2
 OpExecutionMode %1 OriginUpperLeft
-%3 = OpString "$OPSTRING_FILENAME/const-narrowing-cast.rs"
+%3 = OpString "$DIR/const-narrowing-cast.rs"
 OpName %2 "output"
 OpDecorate %2 Location 0
 %4 = OpTypeInt 32 0

tests/compiletests/ui/dis/custom_entry_point.stderr

@@ -2,7 +2,7 @@ OpCapability Shader
 OpMemoryModel Logical Simple
 OpEntryPoint Fragment %1 "hello_world"
 OpExecutionMode %1 OriginUpperLeft
-%2 = OpString "$OPSTRING_FILENAME/custom_entry_point.rs"
+%2 = OpString "$DIR/custom_entry_point.rs"
 OpName %3 "custom_entry_point::main"
 %4 = OpTypeVoid
 %5 = OpTypeFunction %4

tests/compiletests/ui/dis/generic-fn-op-name.stderr

@@ -2,7 +2,7 @@ OpCapability Shader
 OpMemoryModel Logical Simple
 OpEntryPoint Fragment %1 "main"
 OpExecutionMode %1 OriginUpperLeft
-%2 = OpString "$OPSTRING_FILENAME/generic-fn-op-name.rs"
+%2 = OpString "$DIR/generic-fn-op-name.rs"
 OpName %3 "generic_fn_op_name::main"
 OpName %4 "generic_fn_op_name::generic::<f32, {generic_fn_op_name::Dimensionality::TwoD}>"
 %5 = OpTypeVoid