diff options
| author | Nicholas Nethercote <n.nethercote@gmail.com> | 2024-03-21 10:35:19 +1100 |
|---|---|---|
| committer | Nicholas Nethercote <n.nethercote@gmail.com> | 2024-03-21 11:48:49 +1100 |
| commit | 23ee523ea61355c332481566bb5a093f29cdce59 (patch) | |
| tree | 4e5c8b306d5614553b28a978c15bbb6eb6c721c9 /compiler/rustc_interface/src/tests.rs | |
| parent | e3df96cfda905301fc8514e000f942222c1ab6ad (diff) | |
| download | rust-23ee523ea61355c332481566bb5a093f29cdce59.tar.gz rust-23ee523ea61355c332481566bb5a093f29cdce59.zip | |
Remove `CodegenBackend::target_override`.
Backend and target selection is a mess: the target can override the backend (via `Target::default_codegen_backend`), *and* the backend can override the target (via `CodegenBackend::target_override`). The code that handles this is ugly. It calls `build_target_config` twice, once before getting the backend and once again afterward. It also must check that both overrides aren't triggering at the same time. This commit removes the latter override. It's used in rust-gpu but @eddyb said via Zulip that removing it would be ok. This simplifies the code greatly, and will allow some nice follow-up refactorings.
Diffstat (limited to 'compiler/rustc_interface/src/tests.rs')
| -rw-r--r-- | compiler/rustc_interface/src/tests.rs | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/compiler/rustc_interface/src/tests.rs b/compiler/rustc_interface/src/tests.rs index 4d8e749d1da..c796448e5ee 100644 --- a/compiler/rustc_interface/src/tests.rs +++ b/compiler/rustc_interface/src/tests.rs @@ -41,8 +41,7 @@ fn mk_session(matches: getopts::Matches) -> (Session, Cfg) { let sysroot = filesearch::materialize_sysroot(sessopts.maybe_sysroot.clone()); - let target_cfg = - rustc_session::config::build_target_config(&early_dcx, &sessopts, None, &sysroot); + let target_cfg = rustc_session::config::build_target_config(&early_dcx, &sessopts, &sysroot); let sess = build_session( early_dcx, |