[PATCH] D52875: Fix definitions of __builtin_(add|sub|mul)_overflow

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 4 06:23:12 PDT 2018


erichkeane added a comment.

Can you write tests for this please?  Particularly validate the results in a constexpr context.

Additionally, these all have the 't' flag, which means that these signatures are meaningless, right?  What are you seeing where this works incorrectly?


Repository:
  rC Clang

https://reviews.llvm.org/D52875





More information about the cfe-commits mailing list