maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[X86] No need to determine pointer when the type is already a MachineInstr*. NFCI. 

Caught by cppcheck - appears to be a copy+paste typo as the other var is an iterator that does need the &* pointer operation.
This commit is contained in:
Simon Pilgrim 2020-10-31 11:26:25 +00:00
parent 43e451f9f3
commit 5eec049689
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
llvm/lib/Target/X86/X86CallFrameOptimization.cpp
View File

@ -563,8 +563,7 @@ void X86CallFrameOptimization::adjustCallSequence(MachineFunction &MF,
unsigned NumOps = DefMov->getDesc().getNumOperands();
for (unsigned i = NumOps - X86::AddrNumOperands; i != NumOps; ++i)
Push->addOperand(DefMov->getOperand(i));
Push->cloneMergedMemRefs(MF, {&*DefMov, &*Store});
Push->cloneMergedMemRefs(MF, {DefMov, &*Store});
DefMov->eraseFromParent();
} else {
PushOpcode = Is64Bit ? X86::PUSH64r : X86::PUSH32r;