Skip to content

'+fpmr' is not a recognized feature for this target #137924

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
bingmatv opened this issue Mar 3, 2025 · 2 comments
Closed

'+fpmr' is not a recognized feature for this target #137924

bingmatv opened this issue Mar 3, 2025 · 2 comments
Labels
A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. A-target-feature Area: Enabling/disabling target features like AVX, Neon, etc. C-bug Category: This is a bug. O-android Operating system: Android T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@bingmatv
Copy link

bingmatv commented Mar 3, 2025

Host version:

Linux localhost 5.4.249-qgki-28972655-abX218USQS6CYB3 #1 SMP PREEMPT Mon Feb 10 21:56:28 KST 2025 aarch64 Android

When compiling https://crates.io/crates/oilelpqt (manually deleted snmalloc dependency) in release mode, compiler said:

'+fpmr' is not a recognized feature for this target
@bingmatv bingmatv added the C-bug Category: This is a bug. label Mar 3, 2025
@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Mar 3, 2025
@meltyness
Copy link

Likely related to this:

rust/compiler/rustc_codegen_llvm/src/attributes.rs

Line 513 in 81d8edc

// HACK: LLVM versions 19+ do not have the FPMR feature and treat it as always enabled

Perhaps in the interim you can upgrade and use LLVM 19? Or downgrade to a version of LLVM 18 where "+fpmr" option is accepted?

@lolbinarycat lolbinarycat added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Mar 14, 2025
@jieyouxu jieyouxu added A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. O-android Operating system: Android and removed needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. labels Apr 10, 2025
@jieyouxu
Copy link
Member

jieyouxu commented Apr 10, 2025 •
edited
Loading

Triage: minimum external LLVM is now 19 following #139275, +fpmr is not supported because upstream doesn't support it anymore.

@jieyouxu jieyouxu closed this as not planned Won't fix, can't repro, duplicate, stale Apr 10, 2025
@jieyouxu jieyouxu added the A-target-feature Area: Enabling/disabling target features like AVX, Neon, etc. label Apr 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. A-target-feature Area: Enabling/disabling target features like AVX, Neon, etc. C-bug Category: This is a bug. O-android Operating system: Android T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@lolbinarycat @jieyouxu @rustbot @meltyness @bingmatv