[PATCH] D96399: [X86][CodeGenPrepare] Try to reuse IV's incremented value instead of adding the offset

Thu Mar 4 09:56:49 PST 2021

nikic added a comment.

In D96399#2604000 <https://reviews.llvm.org/D96399#2604000>, @reames wrote:

> I went ahead and pushed a change (e0cfd451 <https://reviews.llvm.org/rGe0cfd451718e2524cc5b8f98ecd72a75d37146cc>) to lazily compute domtree as per @nikic's guess.  @xbolva00 do you still see the compile time regression with this change?

Thanks! Here are the results for the range with both commits (it does not look like any other relevant changes landed in the meantime): http://llvm-compile-time-tracker.com/compare.php?from=b15ce2f344ac7845729d2be0a8316b20a32c6292&to=e0cfd451718e2524cc5b8f98ecd72a75d37146cc&stat=instructions

For most programs the regression is resolved, only mafft still has a noteworthy regression, as well as the ReleaseLTO-g configurations (don't know if related to the LTO part or the `-g` part though).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96399/new/

https://reviews.llvm.org/D96399