[PATCH] D52839: Inform AST's UnaryOperator of FENV_ACCESS

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 11 12:14:10 PDT 2018


lebedev.ri added inline comments.


================
Comment at: include/clang/AST/Expr.h:1791
+  // otherwise.
+  unsigned FPFeatures : 3;
+
----------------
Why as the first one, and not after `CanOverflow` ...


================
Comment at: include/clang/AST/Expr.h:1809
+        Opc(opc), CanOverflow(CanOverflow), Loc(l), Val(input)
+        { FPFeatures = FPF.getInt(); }
 
----------------
.. then you should be able to init it in ctor initializer, not in the body.


https://reviews.llvm.org/D52839





More information about the cfe-commits mailing list