[PATCH] D14073: [FPEnv Core 08/14] Do not simplify expressions with FPEnv access

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 2 19:48:23 PST 2016

joker.eph added inline comments.

Comment at: lib/Analysis/ConstantFolding.cpp:1008
@@ -1007,1 +1007,3 @@
+  if (!FMF.any())
+    FMF = CE->getFastMathFlags();
   return ConstantFoldConstantExpressionImpl(CE, DL, TLI, FoldedOps, FMF);
Why not always use `CE->getFastMathFlags();`? Wasn't the `FMF` param added to this function because of the lack of `FMF` in `ConstantExpr` originally? 

Comment at: lib/IR/ConstantsContext.h:423
@@ -422,2 +422,3 @@
   uint16_t SubclassData;
+  FastMathFlags FMF;
   ArrayRef<Constant *> Ops;
I think it is OK, unless you have any clue why it wouldn't?



More information about the llvm-commits mailing list