[llvm] [ARM] Remove `UnsafeFPMath` uses (PR #151275)
David Green via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 25 11:28:19 PDT 2025
================
@@ -222,8 +222,7 @@ void ARMSubtarget::initSubtargetFeatures(StringRef CPU, StringRef FS) {
// NEON f32 ops are non-IEEE 754 compliant. Darwin is ok with it by default.
const FeatureBitset &Bits = getFeatureBits();
if ((Bits[ARM::ProcA5] || Bits[ARM::ProcA8]) && // Where this matters
- (Options.UnsafeFPMath || isTargetDarwin() ||
----------------
davemgreen wrote:
Oh I had meant for these to be part of the other patches - the point was to make this just handle the build attributes (the bit I am less sure about). I missed that in the other patches, I obviously didn't read them carefully enough. The point is to take this patch that does 3 different things are turn it into 3 patches that do 1.
https://github.com/llvm/llvm-project/pull/151275
More information about the llvm-commits
mailing list