[PATCH] D11051: Swap loop invariant GEP with loop variant GEP to allow more LICM.

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 7 06:24:55 PDT 2015


mcrosier added a comment.

In http://reviews.llvm.org/D11051#219017, @atrick wrote:

> Lawrence, Chad, does this pass help you independent from the proposed patch?


I believe we saw a 3-4% regression on Spec200x/mcf with James disabled the pass.  This was on our internal branch in the context of LTO, which departs greatly from the community LTO flow (we're hoping to fix that soon).

> Is it something you will work toward enabling by default. If so, great.


@hulx2000: Let me know if you'd like to discuss this offline.  Have you or Ana discussed your plans with James?


Repository:
  rL LLVM

http://reviews.llvm.org/D11051





More information about the llvm-commits mailing list