diff options
| author | bors <bors@rust-lang.org> | 2024-12-30 12:41:55 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2024-12-30 12:41:55 +0000 |
| commit | f95c9967506712bdf8e86b08d8b17accbde2d985 (patch) | |
| tree | 9c664aa18e06df4785919f1fc96ebfef9a5cb95e /compiler/rustc_codegen_llvm/src | |
| parent | c1566141b6ed24eaa11075aa7dfe6511fdf32231 (diff) | |
| parent | 62bb35ab5d99dc0b7d6cc77687b09e94ccd228c6 (diff) | |
| download | rust-f95c9967506712bdf8e86b08d8b17accbde2d985.tar.gz rust-f95c9967506712bdf8e86b08d8b17accbde2d985.zip | |
Auto merge of #134885 - RalfJung:arm-soft-float, r=workingjubilee
make -Csoft-float have an effect on all ARM targets See the discussion [on Zulip](https://rust-lang.zulipchat.com/#narrow/channel/187780-t-compiler.2Fwg-llvm/topic/Softfloat.20ABI.2C.20hardfloat.20instructions): apparently `-float-abi=soft` is how one can force "use soffloat ABI but hardware FP instructions" on ARM-32. Our version of that is `-Csoft-float` but I made that one a NOP on all targets except for ARM eabihf (see https://github.com/rust-lang/rust/pull/129897)... which now make experimentation on what we actually want to do here a lot harder. So, let's unlock the flag on all ARM targets while we are still figuring out our long-term plans here. Cc `@workingjubilee`
Diffstat (limited to 'compiler/rustc_codegen_llvm/src')
| -rw-r--r-- | compiler/rustc_codegen_llvm/src/back/write.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/compiler/rustc_codegen_llvm/src/back/write.rs b/compiler/rustc_codegen_llvm/src/back/write.rs index 45294ea35b1..ae5e818d86f 100644 --- a/compiler/rustc_codegen_llvm/src/back/write.rs +++ b/compiler/rustc_codegen_llvm/src/back/write.rs @@ -189,7 +189,7 @@ pub(crate) fn target_machine_factory( let reloc_model = to_llvm_relocation_model(sess.relocation_model()); let (opt_level, _) = to_llvm_opt_settings(optlvl); - let use_softfp = if sess.target.arch == "arm" && sess.target.abi == "eabihf" { + let use_softfp = if sess.target.arch == "arm" { sess.opts.cg.soft_float } else { // `validate_commandline_args_with_session_available` has already warned about this being |