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

[Mehdi AMINI via llvm-commits]

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?


Repository:
  rL LLVM

http://reviews.llvm.org/D14073