[PATCH] D48053: Correct behavior of __builtin_*_overflow and constexpr.

[Eli Friedman via Phabricator via cfe-commits]

efriedma added a reviewer: rsmith.
efriedma added inline comments.


================
Comment at: lib/Sema/SemaChecking.cpp:228
+          S.getASTContext(), Ty, /*consume*/ false);
+      Arg = S.PerformCopyInitialization(Entity, SourceLocation(), Arg);
+      TheCall->setArg(2, Arg.get());
----------------
Is it possible for this initialization to fail?  If it can't fail, please add an assertion; otherwise, you probably need to "return true".  (Not sure off the top of my head how that would happen; maybe it would error out on a `volatile int*`?)


Repository:
  rC Clang

https://reviews.llvm.org/D48053