[PATCH] D123394: [CodeGen] Late cleanup of redundant address/immediate definitions.

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D123394#3439949 <https://reviews.llvm.org/D123394#3439949>, @efriedma wrote:

> Have you compared your approach to LocalStackSlotAllocation?  That's doing something sort of similar, but earlier in the pass pipeline.  Not sure if that covers exactly the same cases as this patch.

The problem I've run into is with large frame offsets for spill slots, which don't exist during LocalStackSlotAllocation when things are easy to do. PEI sees each spill individually and you need to materialize some large offset for each instance.


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

https://reviews.llvm.org/D123394