[llvm] [DAG] Don't split f64 constant stores if the fp imm is legal (PR #74622)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 6 20:19:16 PST 2023


================
@@ -20940,7 +20940,7 @@ SDValue DAGCombiner::replaceStoreOfFPConstant(StoreSDNode *ST) {
                           Ptr, ST->getMemOperand());
     }
 
-    if (ST->isSimple() &&
+    if (ST->isSimple() && !TLI.isFPImmLegal(CFP->getValueAPF(), MVT::f64) &&
         TLI.isOperationLegalOrCustom(ISD::STORE, MVT::i32)) {
----------------
arsenm wrote:

Maybe add this as the last condition?

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


More information about the llvm-commits mailing list