Previously, we only added early-clobber flags to the 'group' immediate flag operand
of an inline asm operand.
However, we also have to add the EarlyClobber flag to the MachineOperand itself.
This fixes PR46028
Paths
| Differential D80467
[GlobalISel][InlineAsm] Add missing EarlyClobber flag to inline asm output operands ClosedPublic Authored by kschwarz on May 23 2020, 5:19 AM.
Details Summary Previously, we only added early-clobber flags to the 'group' immediate flag operand This fixes PR46028
Diff Detail
Event TimelineThis revision is now accepted and ready to land.May 23 2020, 8:33 AM Comment Actions LGTM. Confirmed the test passes after this and the correct assembly is generated. Thanks for the fix! Closed by commit rGf2fad3f703aa: [GlobalISel][InlineAsm] Add missing EarlyClobber flag to inline asm output… (authored by kschwarz). · Explain WhyMay 27 2020, 3:12 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 266471 llvm/lib/CodeGen/GlobalISel/InlineAsmLowering.cpp
llvm/test/CodeGen/AArch64/GlobalISel/irtranslator-inline-asm.ll
|