[PATCH] D62653: [Mips][DSP] Fix physregs incorrectly marked as dead.

Mirko Brkusanin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 27 09:11:28 PDT 2019


mbrkusanin marked an inline comment as done.
mbrkusanin added inline comments.


================
Comment at: lib/Target/Mips/MipsSEISelDAGToDAG.cpp:158
+  Reg = MipsRegisterInfo::getDSPOutFlagIfSubreg(Reg);
+  MO.setIsDead();
+  DeadImplRegs[Reg] = &MO;
----------------
efriedma wrote:
> Does this setIsDead() call actually run for some testcase?  I don't see any dead flag in your testcase.
You're right. We don't need it anymore. This was needed in an older version of the patch. Now we can just check if it's dead.


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

https://reviews.llvm.org/D62653





More information about the llvm-commits mailing list