[PATCH] D54128: Fix MachineInstr::findRegisterUseOperandIdx subreg checks

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 5 21:03:38 PST 2018


rampitec updated this revision to Diff 172709.
rampitec edited the summary of this revision.
rampitec removed reviewers: junbuml, sbaranga, sdardis.
rampitec added a comment.

I have realized the old check was also needed, an instruction reads a register in both cases: if the operand is a subreg of a requested register or if it is a superreg.

With the new patch previous changes to other targets are gone. Therefor gone the ask to review their test changes.


https://reviews.llvm.org/D54128

Files:
  lib/CodeGen/MachineInstr.cpp
  test/CodeGen/AMDGPU/optimize-if-exec-masking.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54128.172709.patch
Type: text/x-patch
Size: 3616 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181106/0b29dced/attachment.bin>


More information about the llvm-commits mailing list