rjmccall added inline comments.

================
Comment at: clang/lib/AST/ExprConstant.cpp:9839
+    return Success(Result, E);
+  }
+
----------------
Can most of this reasonably be a method on `APFixedPoint`?  (And likewise for 
`CK_IntegralToFixedPoint`.)  If nothing else, I suspect you are going to want 
these when it comes to things like the compound assignment operators, which do 
implicit conversions internally that aren't directly expressed in the AST.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56900/new/

https://reviews.llvm.org/D56900



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to