diff --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst index 9bbf270c27476..6f582f584bdf6 100644 --- a/clang/docs/ReleaseNotes.rst +++ b/clang/docs/ReleaseNotes.rst @@ -476,6 +476,8 @@ Bug Fixes in This Version - Fix crash when passing a braced initializer list to a parentehsized aggregate initialization expression. (`#63008 `_). +- Reject increment of bool value in unevaluated contexts after C++17. + (`#47517 `_). Bug Fixes to Compiler Builtins ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ diff --git a/clang/include/clang/Basic/DiagnosticSemaKinds.td b/clang/include/clang/Basic/DiagnosticSemaKinds.td index 90ecbd623ceef..5895888c175fa 100644 --- a/clang/include/clang/Basic/DiagnosticSemaKinds.td +++ b/clang/include/clang/Basic/DiagnosticSemaKinds.td @@ -7661,7 +7661,7 @@ def warn_increment_bool : Warning< "incompatible with C++17">, InGroup; def ext_increment_bool : ExtWarn< "ISO C++17 does not allow incrementing expression of type bool">, - DefaultError, InGroup; + DefaultError, SFINAEFailure, InGroup; def err_increment_decrement_enum : Error< "cannot %select{decrement|increment}0 expression of enum type %1">; diff --git a/clang/test/SemaCXX/bool-increment-SFINAE.cpp b/clang/test/SemaCXX/bool-increment-SFINAE.cpp new file mode 100644 index 0000000000000..d3889293fc0b6 --- /dev/null +++ b/clang/test/SemaCXX/bool-increment-SFINAE.cpp @@ -0,0 +1,37 @@ +// RUN: %clang_cc1 %std_cxx98-14 -fsyntax-only -verify=precxx17 %s +// RUN: %clang_cc1 -std=c++17 -fsyntax-only -verify=expected %s +// RUN: %clang_cc1 -std=c++17 -DFAILED_CXX17 -fsyntax-only -verify=failcxx17 %s +// RUN: %clang_cc1 %std_cxx20- -fsyntax-only -verify=cxx20 %s +// expected-no-diagnostics + +template auto f(T t) -> decltype(++t); // precxx17-warning {{incrementing expression of type bool is deprecated}} + +auto f(...) -> void; +void g() { f(true); } + +#ifdef FAILED_CXX17 + +template auto f1(T t) -> decltype(++t); // failcxx17-note {{candidate template ignored: substitution failure [with T = bool]: ISO C++17 does not allow incrementing expression of type bool}} +auto f1(void) -> void; // failcxx17-note {{candidate function not viable: requires 0 arguments, but 1 was provided}} +void g1() { f1(true); } // failcxx17-error {{no matching function for call to 'f1'}} + +#endif + +#if __cplusplus >= 202002L +template +concept can_increment = requires(T t) { + ++t; +}; + +template +void f() { + static_assert(requires(T t) { ++t; }); // cxx20-error {{static assertion failed due to requirement 'requires (bool t) { <>; }'}} +} + +int main() { + f(); // cxx20-note {{in instantiation of function template specialization 'f' requested here}} + static_assert(!can_increment); + + return 0; +} +#endif