[PATCH] D110238: [LiveIntervals] Fix repairOldRegInRange for simple def cases

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 22 08:39:20 PDT 2021


arsenm added a comment.

In D110238#3015149 <https://reviews.llvm.org/D110238#3015149>, @foad wrote:

>> I don't follow how the test changes stress this though
>
> -early-live-intervals means that LiveIntervals is available to TwoAddressInstructionPass, which uses repairIntervalsInRange to update it. Without this patch we hit various assertion and/or verification failures because it doesn't get updated correctly.

Can you add a dedicated MIR pass that just runs TwoAddressInstructionPass then?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110238



More information about the llvm-commits mailing list