erichkeane wrote:

> > Why doesn't this result in any changes to tests? I would expect having to 
> > remove them, right?
> 
> There was a single test that was relying on sign mismatch, and then during 
> code review we made the diagnostic to ignore signness. So it ended up merged 
> without test case where this diagnostic was triggered.

Ooof, that is disappointing.  We should have never let this in without test 
coverage.

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

Reply via email to