diff options
| author | Ralf Jung <post@ralfj.de> | 2024-09-29 12:16:35 +0200 |
|---|---|---|
| committer | Ralf Jung <post@ralfj.de> | 2024-09-29 12:16:35 +0200 |
| commit | a78fd694d47608eb382478b038ad0e91b6f5150d (patch) | |
| tree | 6ce14f128caceffead8f203412e6e13897691f47 /compiler/rustc_codegen_llvm/src/llvm_util.rs | |
| parent | 9903b256a2f5d57a912c6aa793d4373db30f9f9a (diff) | |
| download | rust-a78fd694d47608eb382478b038ad0e91b6f5150d.tar.gz rust-a78fd694d47608eb382478b038ad0e91b6f5150d.zip | |
extend comment in global_llvm_features regarding target-cpu=native handling
Diffstat (limited to 'compiler/rustc_codegen_llvm/src/llvm_util.rs')
| -rw-r--r-- | compiler/rustc_codegen_llvm/src/llvm_util.rs | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/compiler/rustc_codegen_llvm/src/llvm_util.rs b/compiler/rustc_codegen_llvm/src/llvm_util.rs index a2a5499597c..201cfbb1918 100644 --- a/compiler/rustc_codegen_llvm/src/llvm_util.rs +++ b/compiler/rustc_codegen_llvm/src/llvm_util.rs @@ -536,6 +536,11 @@ pub(crate) fn global_llvm_features( // -Ctarget-cpu=native match sess.opts.cg.target_cpu { Some(ref s) if s == "native" => { + // We have already figured out the actual CPU name with `LLVMRustGetHostCPUName` and set + // that for LLVM, so the features implied by that CPU name will be available everywhere. + // However, that is not sufficient: e.g. `skylake` alone is not sufficient to tell if + // some of the instructions are available or not. So we have to also explicitly ask for + // the exact set of features available on the host, and enable all of them. let features_string = unsafe { let ptr = llvm::LLVMGetHostCPUFeatures(); let features_string = if !ptr.is_null() { |