[PATCH] D11854: DeadStoreElimination: remove a redundant store even if the load is in a different block.

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 8 06:37:55 PDT 2015


dberlin added a subscriber: dberlin.
dberlin added a comment.

So, Karthik posted and got http://reviews.llvm.org/D11143 reviewed

Does this not cover your case?

Additionally, "            MD->getNonLocalPointerDependency(SI, Deps);"

This is very compile time expensive to call a lot. I'd really like to know how often yours actually triggers, and what performance different it actually makes relative to Karthik's


http://reviews.llvm.org/D11854





More information about the llvm-commits mailing list