adamcz added inline comments.
================
Comment at: clang/test/SemaCXX/concept-fatal-error.cpp:8
+ // We test that we do not crash in such cases (#55401)
+ int i = requires { { i } f } // expected-error {{expected ';' at end of
declaration list}}
+
----------------
ilya-biryukov wrote:
> We need want to pass `-verify` to clang for these directives to have an
> effect.
> The fatal error can be matched by `// expected-error@* {{too...}}` (maybe
> that's obvious, but it took me some time to figure it out first time I needed
> it)
> And there is no need to run `FileCheck` after that!
Ah, thanks! I knew there was a way to match fatal errors, I just didn't know
what it was.
No clue what happened to -verify, I swear it was there at some point ;-)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129499/new/
https://reviews.llvm.org/D129499
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
