[llvm] r309821 - [AArch64] Simplify AES*Tied pseudo expansion (NFC).

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 3 10:01:11 PDT 2017


Hi Florian,

On 2 August 2017 at 08:17, Florian Hahn via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> [AArch64] Simplify AES*Tied pseudo expansion (NFC).
>
> Summary:
> Suggested by @t.p.northover in https://bugs.llvm.org/show_bug.cgi?id=34015.

Sorry about this, but it seems I was wrong. If we just call setDesc
then the MachineOperands are still tied and the verifier gets confused
(leading to at least one Green Dragon failure). I couldn't find a
simple way to untie them so I've just reverted this in r309962.

I don't think there's any need for a followup to reapply this, it was
a peephole change anyway really.

Cheers.

Tim.


More information about the llvm-commits mailing list