[PATCH] D30352: [JumpThread] Use AA in SimplifyPartiallyRedundantLoad()

Jun Bum Lim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 28 13:41:57 PST 2017


junbuml added a comment.

My original intension for this change was to handle a missing opportunity that should be handled in here so that it can obviously open up more threading, which I believe match with the purpose of SimplifyPartiallyRedundantLoad() in jump thread. Regarding why such pattern is exposed in this pass, I can add another test case by reducing from the real case like https://reviews.llvm.org/D29571 for my previous patch. Would it be acceptable with you?


https://reviews.llvm.org/D30352





More information about the llvm-commits mailing list