This is an archive of the discontinued LLVM Phabricator instance.

Add baseline tests for D147404
ClosedPublic

Authored by junaire on Apr 2 2023, 2:36 AM.

Details

Summary

Signed-off-by: Jun Zhang <jun@junz.org>

Diff Detail

Event Timeline

junaire created this revision.Apr 2 2023, 2:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 2 2023, 2:36 AM
junaire requested review of this revision.Apr 2 2023, 2:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 2 2023, 2:36 AM

Sounds good. Can you add these to the existing hadd-combine.ll file instead, and add aarch64.neon.shadd/uhadd/urhadd variants too? AArch64 uses hadd/rhadd names, as opposed to avg like X86.

llvm/test/CodeGen/AArch64/avg.ll
22 ↗(On Diff #510327)

v2i16 -> v2i32

junaire updated this revision to Diff 510337.Apr 2 2023, 4:59 AM

Add all rest variants

Thanks for adding the extra tests. Do you mind moving them to llvm/test/CodeGen/AArch64/hadd-combine.ll, so that they are with the other combine tests we have.

junaire updated this revision to Diff 510427.Apr 3 2023, 1:54 AM

Address @dmgreen's comments, thanks!

dmgreen accepted this revision.Apr 3 2023, 1:56 AM

Thanks. LGTM

This revision is now accepted and ready to land.Apr 3 2023, 1:56 AM
This revision was landed with ongoing or failed builds.Apr 3 2023, 1:58 AM
This revision was automatically updated to reflect the committed changes.