[PATCH] D61671: X86WinAllocaExpander: Drop code looking through register copies (PR41786)
Hans Wennborg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 8 04:38:38 PDT 2019
hans added inline comments.
================
Comment at: llvm/lib/Target/X86/X86WinAllocaExpander.cpp:269
+ if (MRI->use_empty(AmountReg)) {
MachineInstr *AmountDef = MRI->getUniqueVRegDef(AmountReg);
AmountDef->eraseFromParent();
----------------
RKSimon wrote:
> Can this be null? We were testing for it before and still do above.
Yes, we should keep that. Good catch.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61671/new/
https://reviews.llvm.org/D61671
More information about the llvm-commits
mailing list