fhahn added a comment. It looks like a recently introduced GVN crash is exposed by this change (probably D87061 <https://reviews.llvm.org/D87061>). I will commit once this is fixed.
In D87163#2263685 <https://reviews.llvm.org/D87163#2263685>, @nikic wrote: > LGTM as well, per llvm-dev discussion. I think you've done all the due > diligence we can expect. > > Do you plan to also work on MSSA-based MemCpyOpt? I expect that one will have > a large positive impact on Rust code, where lack of cross-BB memcpy elision > is a big issue right now. I probably won't have time to work on MSSA-based MemCpyOpt for the next couple of weeks unfortunately. But I think there is a lot that can be re-used/shared from the DSE implementation. Happy to discuss approaches. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87163/new/ https://reviews.llvm.org/D87163 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
