[PATCH] D126273: [DAGCombiner][VP] Add DAGCombine for merging VP_FADD and VP_FMUL to VP_FMA.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:23055
+      (Options.AllowFPOpFusion == FPOpFusion::Fast || Options.UnsafeFPMath);
+
+  // Is the node an VP_FMUL and contractable either due to global flags or
----------------
What about these early exits from the original function

```
  // If the addition is not contractable, do not combine.                        
  if (!AllowFusionGlobally && !N->getFlags().hasAllowContract())                 
    return SDValue();                                                            
                                                                                 
  if (TLI.generateFMAsInMachineCombiner(VT, OptLevel))                           
    return SDValue(); 
```


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126273/new/

https://reviews.llvm.org/D126273