[llvm] [AArch64] Remove `UnsafeFPMath` (PR #150876)

via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 28 03:49:48 PDT 2025


================
@@ -12292,7 +12290,8 @@ SDValue AArch64TargetLowering::getSqrtEstimate(SDValue Operand,
       SDLoc DL(Operand);
       EVT VT = Operand.getValueType();
 
-      SDNodeFlags Flags = SDNodeFlags::AllowReassociation;
+      SDNodeFlags Flags =
----------------
paperchalice wrote:

Reassoc flag is not enough, aarch64 backend requires nsz to generate expected code, see the change in llvm/lib/Target/AArch64/AArch64InstrInfo.cpp. IMO extra nsz flag seems too strict here.
Will add related comment later. 

https://github.com/llvm/llvm-project/pull/150876


More information about the llvm-commits mailing list