xazax.hun added a comment.

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.


https://reviews.llvm.org/D5767



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to