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

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 28 11:59:23 PST 2017


dberlin added a comment.

In the future, can you please analyze why this is happening?
This is another example of a test case that is unrelated to the pattern you are trying to optimize.  That means if this broke optimizing this tomorrow, we'd have to run spec again and analyze it.  
We also really need to understand what optimizations need to be happening, not just randomly implement stuff in various passes because it increases SPEC


https://reviews.llvm.org/D30352





More information about the llvm-commits mailing list