boomanaiden154 wrote:

> LGTM. May this land prior to LLVM 22 branching?

I'd prefer to wait given the branch date is in a couple of days. This patch 
will involve a let of test churn (attribute gets reordered in tests now that it 
has been adjusted), which might backporting slightly more difficult. It's also 
not useful until the DSE/Clang changes land, and they're both not through the 
review process yet.

Thank you for putting together the Alive2 changes for this and for the reviews!

@nikic When you get a chance, do you mind taking a look and letting me know if 
you have any concerns?

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

Reply via email to