AaronBallman wrote:

> This is a lot of tests... do we really need every combination of 
> type/operator/flag? I guess it makes sure we don't have gaps, but thousands 
> of lines of tests seems excessive.

It does seem like a lot of testing, but I'd rather we be excessive than take 
our historical approach of only getting a fraction of test coverage. (FWIW, it 
would be nice if we could get coverage reports for testing in Clang and maybe 
even hook it up to precommit CI so that we reject patches which reduce test 
coverage ratios rather than increase them, but that's not an issue for this PR.)

https://github.com/llvm/llvm-project/pull/131447
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to