Szelethus added a comment. In https://reviews.llvm.org/D5767#1000347, @xazax.hun wrote:
> In https://reviews.llvm.org/D5767#999143, @sabel83 wrote: > > > 2. What do you mean by regression tests? We have run the clang-test target > > successfully on the patched code (which has the hook). Note that the hook > > this pull request provides is implemented as a ProgramAction. It is not > > intended to be used together with other program actions, I don't see how we > > could turn it on for other tests (if that is what you referred to). > > > I would bet the sema tests are full of tricky edge cases. So running > templight on those tests would be a helpful exercise. I am only interested in > assertion fails, so we do not need to check the output. One option to do so > would be to add the -templight-dump option to the %clang_cc1 variable when > running the tests. Note that the tests are likely to fail because the output > will change, but if there are no crashes, it is fine. I did just that, and nothing crashed during the regression tests (in fact, every single one passed)! https://reviews.llvm.org/D5767 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
