[PATCH] D124050: llvm-reduce: Fix not removing first instruction in MachineBasicBlock

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 19 15:43:15 PDT 2022


arsenm created this revision.
arsenm added reviewers: markus, reames, aeubanks, qcolombet, MatzeB.
Herald added a project: All.
arsenm requested review of this revision.
Herald added a subscriber: wdng.
Herald added a project: LLVM.

This had the surprising behavior of using whatever instruction
happened to be first in the block as an anchor point to stick random
implicit defs on. Use a real implicit_def instead.


https://reviews.llvm.org/D124050

Files:
  llvm/test/tools/llvm-reduce/mir/generic-vreg.mir
  llvm/test/tools/llvm-reduce/mir/instr-reduce.mir
  llvm/test/tools/llvm-reduce/mir/preserve-frame-info.mir
  llvm/test/tools/llvm-reduce/mir/subreg-def0.mir
  llvm/test/tools/llvm-reduce/mir/subreg-def1.mir
  llvm/tools/llvm-reduce/deltas/ReduceInstructionsMIR.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D124050.423753.patch
Type: text/x-patch
Size: 6540 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220419/2ca92a9d/attachment.bin>


More information about the llvm-commits mailing list