This is an archive of the discontinued LLVM Phabricator instance.

[AArch64] Replace uaddlv intrinsic with uaddlv sdnode
ClosedPublic

Authored by jaykang10 on Sep 5 2023, 7:07 AM.

Details

Summary

We have introduced a new custom SDNode UADDLV on https://reviews.llvm.org/D159267.
Let's try to replace uaddlv intrinsic with it.

Diff Detail

Event Timeline

jaykang10 created this revision.Sep 5 2023, 7:07 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 5 2023, 7:07 AM
jaykang10 requested review of this revision.Sep 5 2023, 7:07 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 5 2023, 7:07 AM
dmgreen added inline comments.Sep 5 2023, 8:52 AM
llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
8760

Here too!

jaykang10 added inline comments.Sep 5 2023, 9:11 AM
llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
8760

You are right!
Let me update it too.

jaykang10 updated this revision to Diff 555877.Sep 5 2023, 9:21 AM

Replaced one more case.

dmgreen accepted this revision.Sep 5 2023, 9:29 AM

Thanks. LGTM

This revision is now accepted and ready to land.Sep 5 2023, 9:29 AM
This revision was landed with ongoing or failed builds.Sep 5 2023, 9:58 AM
This revision was automatically updated to reflect the committed changes.