[PATCH] D48040: Implement constexpr __builtin_*_overflow

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 11 12:31:10 PDT 2018


efriedma added inline comments.


================
Comment at: lib/Sema/SemaChecking.cpp:210
+    Arg = S.PerformCopyInitialization(Entity, SourceLocation(), Arg);
+    TheCall->setArg(I, Arg.get());
   }
----------------
Can you split this change into a separate patch?  Testcase:

```
int a() {
  const int x = 3;
  static int z;
  constexpr int * y = &z;
  return []() { return __builtin_sub_overflow(x,x,y); }();
}```


https://reviews.llvm.org/D48040





More information about the cfe-commits mailing list