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
