[PATCH] D72621: PR44514: Fix recovery from noexcept with non-convertible expressions

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 13 14:02:07 PST 2020


erichkeane marked an inline comment as done.
erichkeane added inline comments.


================
Comment at: clang/lib/Sema/SemaExceptionSpec.cpp:90-92
+    llvm::APSInt Value{1};
+    Value = 0;
+    return ConstantExpr::Create(Context, BoolExpr, APValue{Value});
----------------
aaron.ballman wrote:
> This can be simplified with `APSInt::getUnsigned()` unless you reallllly need it to be a one-bit integer.
We normally DO create it as a 1 bit APSInt (the return type of the ConstantExpr), as discussed on IRC we should probably not differ from that.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72621





More information about the cfe-commits mailing list