This is an archive of the discontinued LLVM Phabricator instance.

[AArch64][SVE] Extend PTEST_ANY(X=OP(PG,...), X) -> PTEST_ANY(PG, X)) instcombine
ClosedPublic

Authored by c-rhodes on Oct 21 2022, 3:55 AM.

Details

Summary

Extend above instcombine added in D134946 to cover more flag-setting
instructions.

Diff Detail

Event Timeline

c-rhodes created this revision.Oct 21 2022, 3:55 AM
c-rhodes requested review of this revision.Oct 21 2022, 3:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 21 2022, 3:55 AM
Matt added a subscriber: Matt.Oct 21 2022, 12:37 PM
peterwaller-arm accepted this revision.Nov 1 2022, 9:05 AM

LGTM, with one minor nit.

llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
1001

I see an extra ) on the right there, which I guess also made it into the commit title and the tests.

llvm/test/Transforms/InstCombine/AArch64/sve-intrinsics-ptest.ll
71

Nit: Extraneous ).

This revision is now accepted and ready to land.Nov 1 2022, 9:05 AM