xakep8 wrote:

> I think this is essentially removing the assert, without a reasonable 
> explanation. So please make sure you disclose AI use in the PR message, 
> thanks.

Yes sorry for just the conclusion in the PR description and not the explanation 
of the fix and what made me make the decision, I've posted the explanation in 
https://github.com/llvm/llvm-project/pull/237969#discussion_r4445118378 I'll 
also add it to the PR description.

https://github.com/llvm/llvm-project/pull/237969
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to