[PATCH] D38342: [C++] Parse (sub) postfix expression after boolean literal

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 28 13:59:40 PDT 2017


aaron.ballman added a comment.

In https://reviews.llvm.org/D38342#883872, @Rakete1111 wrote:

> - Moved test to test/CXX/
>
>   Do I actually need to -verify the test if no diagnostics are expected?
>
>   Thanks @aaron.ballman


Yes, you need to make sure that verify is on the RUN line. The existing test is broken like that (good catch). You should add -verify to the run line and // expected-no-diagnostics below it.


https://reviews.llvm.org/D38342





More information about the cfe-commits mailing list