Displaying 2 results from an estimated 2 matches for "haslivedef".
2017 Jun 27
4
Ok with mismatch between dead-markings in BUNDLE and bundled instructions?
...assert(MI && "No instruction defining live value");
MI->addRegisterDead(LI.reg, TRI);
and the reload after the BUNDLE is introduced by
InlineSpiller::spillAroundUses which has determined that the register is
live:
// Rewrite instruction operands.
bool hasLiveDef = false;
for (const auto &OpPair : Ops) {
MachineOperand &MO = OpPair.first->getOperand(OpPair.second);
MO.setReg(NewVReg);
if (MO.isUse()) {
if (!OpPair.first->isRegTiedToDefOperand(OpPair.second))
MO.setIsKill();
} else {...
2017 Jun 29
2
Ok with mismatch between dead-markings in BUNDLE and bundled instructions?
> On Jun 28, 2017, at 5:10 PM, Quentin Colombet via llvm-dev <llvm-dev at lists.llvm.org> wrote:
>
> Oh wait, vreg1 is indeed used.
> Yeah, having a dead flag here sounds wrong.
I mean on the instruction itself.
On the bundle, that’s debatable. That would fit the semantic “if no side effect you can kill it” (here there is side effect, we define other vregs).
>
>> On