This is an archive of the discontinued LLVM Phabricator instance.

Summary: Teach the AArch64 backend patterns to generate the RAX1 instruction.
ClosedPublic

Authored by BK1603 on Apr 9 2023, 10:34 AM.

Diff Detail

Event Timeline

BK1603 created this revision.Apr 9 2023, 10:34 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 9 2023, 10:34 AM
BK1603 requested review of this revision.Apr 9 2023, 10:34 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 9 2023, 10:34 AM

This looks like a good patch. It looks like the two operands for rax1 may be the wrong way around though.

BK1603 updated this revision to Diff 512197.Apr 10 2023, 10:52 AM

D147887: Summary: Teach the AArch64 backend patterns to generate the RAX1 instruction.

@dmgreen

Fixed! Please review.

dmgreen accepted this revision.Apr 10 2023, 11:39 PM

Thanks. LGTM

This revision is now accepted and ready to land.Apr 10 2023, 11:39 PM

Hi @dmgreen

I don't think I have commit access to the repo, so you might have to commit this patch as well. Thanks!

Can do. Do you have a "Name <email@domain.com>" I can use for attribution? Thanks

HI @dmgreen

You can use
Shreyansh Chouhan <chouhan.shreyansh2702@gmail.com>