[llvm] Update foldFMulReassoc to respect absent fast-math flags (PR #88589)

Andy Kaylor via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 16 14:08:36 PDT 2024


================
@@ -311,6 +312,31 @@ class BinaryOperator : public Instruction {
     return BO;
   }
 
+  static BinaryOperator *
+  CreateWithFMF(BinaryOps Opc, Value *V1, Value *V2, FastMathFlags FMF,
+                const Twine &Name = "", Instruction *InsertBefore = nullptr) {
+    BinaryOperator *BO = Create(Opc, V1, V2, Name, InsertBefore);
----------------
andykaylor wrote:

I don't think so. The include is needed because I'm passing FastMathFlags by value. It gets passed as a single integer, so it doesn't really make sense to pass by pointer.

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


More information about the llvm-commits mailing list