zyn0217 wrote:

> I'm reducing the test case. In the meantime, should this be reverted?

I think this depends on the case. Note the issues addressed by this patch may 
not only cause some crashes in debug build, but probably lead to incorrect 
constraint evaluation in release build, which means there's possibility that 
your case was an accept-invalid before.

(Sorry for making confusion earlier; my office hour just begins and I'm sitting 
here for your case. If there is really something wrong caused by this PR, I'll 
revert it in time. Don't worry.)

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

Reply via email to