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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
