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

Sergey Dmitrouk via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 21 02:50:45 PST 2015


sdmitrouk marked an inline comment as done.

================
Comment at: lib/IR/ConstantsContext.h:423
@@ -422,2 +422,3 @@
   uint16_t SubclassData;
+  FastMathFlags FMF;
   ArrayRef<Constant *> Ops;
----------------
joker.eph wrote:
> Can we reuse one of the previous field?
In case of binary expressions `SubclassData` holds `0` and `SubclassOptionalData` holds binary operation specific flags. We can use `SubclassData`, which will be inconsistent with other types of constant expressions, but can be done if that's OK.


Repository:
  rL LLVM

http://reviews.llvm.org/D14073





More information about the llvm-commits mailing list