search for: setisdead

...ple duplicate COPYs are being generated, why? One for each use? The reg_bundle holds these multiple copies so that we are iterating over the same exact COPY instructions multiple times, even though each one hits the same spot in the function: if (hoistSpill(OldLI, MI)) { MI->getOperand(0).setIsDead(); DeadDefs.push_back(MI); continue; } Even when I add a check: if (MI->getOperand(0)->isDead()) continue; This never checks to true, on the second, third, fourth, etc duplicate. Not sure what I'm missing? Can someone give me a brief description of why these multiple COP...

...is not appropriate. Cheers, -Quentin > > The reg_bundle holds these multiple copies so that we are iterating over the same exact COPY instructions multiple times, even though each one hits the same spot in the function: > > if (hoistSpill(OldLI, MI)) { > MI->getOperand(0).setIsDead(); > DeadDefs.push_back(MI); > continue; > } > > Even when I add a check: > > if (MI->getOperand(0)->isDead()) > continue; > > This never checks to true, on the second, third, fourth, etc duplicate. > > Not sure what I'm missing? Can s...

...ePointerElim(MF) || + return (MF.getTarget().Options.DisableFramePointerElim(MF) || SaveArgs || TRI->needsStackRealignment(MF) || MFI.hasVarSizedObjects() || MFI.isFrameAddressTaken() || MFI.hasOpaqueSPAdjustment() || @@ -850,6 +851,25 @@ MI->getOperand(3).setIsDead(); } +// FIXME: Get this from tablegen. +static ArrayRef<MCPhysReg> get64BitArgumentGPRs(CallingConv::ID CallConv, + const X86Subtarget &Subtarget) { + assert(Subtarget.is64Bit()); + + if (Subtarget.isCallingConvWin64(CallConv)) { + s...

Patch for review. On Mon, Aug 21, 2017 at 11:45 PM Puyan Lotfi <puyan.lotfi.llvm at gmail.com> wrote: > Ping. > > Still working on preparing code for review. Will have a patch for review > ready in the coming days. > > PL > > On Tue, Aug 15, 2017 at 12:06 PM Puyan Lotfi <puyan.lotfi.llvm at gmail.com> > wrote: > >> Hi, >> >> >>