This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add more .vx patterns for VLMax integer setccs.
ClosedPublic

Authored by craig.topper on Apr 6 2022, 2:02 PM.

Details

Summary

This patch synchronizes the structure of the templates with those
in RISCVInstrInfoVVLPatterns.td so that we get patterns with .vx
on the left hand side.

Diff Detail

Event Timeline

craig.topper created this revision.Apr 6 2022, 2:02 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 6 2022, 2:02 PM
craig.topper requested review of this revision.Apr 6 2022, 2:02 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 6 2022, 2:02 PM
craig.topper added inline comments.Apr 6 2022, 2:36 PM
llvm/test/CodeGen/RISCV/rvv/cttz-sdnode.ll
53

This was bad canonicalization of a splat with 0 which should stay on the RHS. I've posted D123256 to fix the canonicalization.

rogfer01 accepted this revision.Apr 6 2022, 11:07 PM

LGTM. Thanks for the cleanup @craig.topper !

This revision is now accepted and ready to land.Apr 6 2022, 11:07 PM
This revision was landed with ongoing or failed builds.Apr 7 2022, 9:22 AM
This revision was automatically updated to reflect the committed changes.