[PATCH] D119070: [MachineCombiner] Update iterator while deleting instructions

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D119070#3299167 <https://reviews.llvm.org/D119070#3299167>, @DarshanRamakant wrote:

> I did not find any existing test case which specifically tests the "insertDeleteInstructions()" function. Also, I did not find MIR test cases with many of the previous commits. Please provide me with some pointers on testing this specific change. I found this issue when experimenting with this transform for my company-specific backend.

I would hope there were at least some IR tests in previous commits touching this pass. Perhaps you can use -stop-after to get a MIR starting point on one of them, and manually modify the MIR until it breaks?


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

https://reviews.llvm.org/D119070