squiid/rust
1
0
Fork 0
Code Issues Pull requests Projects Activity

Remove duplicate aliases for codegen_{cranelift,gcc}

Browse source 
Bootstrap already allows selecting these in `PathSet::has`, which allows
any string that matches the end of a full path.

I found these by adding `assert!(path.exists())` in `StepDescription::paths`.
I think ideally we wouldn't have any aliases that aren't paths, but I've held
off on enforcing that here since it may be controversial, I'll open a separate PR.
This commit is contained in:
Joshua Nelson 2022-04-10 16:37:24 -05:00
parent 027a232755
commit 986c1687f8
1 changed files with 1 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/bootstrap/check.rs
View file

@ -243,12 +243,7 @@ impl Step for CodegenBackend {
const DEFAULT: bool = true; const DEFAULT: bool = true;
fn should_run(run: ShouldRun<'_>) -> ShouldRun<'_> { fn should_run(run: ShouldRun<'_>) -> ShouldRun<'_> {
run.paths(&[ run.paths(&["compiler/rustc_codegen_cranelift", "compiler/rustc_codegen_gcc"])
"compiler/rustc_codegen_cranelift",
"rustc_codegen_cranelift",
"compiler/rustc_codegen_gcc",
"rustc_codegen_gcc",
])
} }
fn make_run(run: RunConfig<'_>) { fn make_run(run: RunConfig<'_>) {