This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Don't reduce vslidedown's VL in rotations
ClosedPublic

Authored by luke on May 24 2023, 4:54 PM.

Details

Summary

Even though we only need to write to the bottom NumElts - Rotation
elements for the vslidedown.vi, we can save an extra vsetivli toggle if
we just keep the wide VL.

(I may be missing something here: is there a reason why we want to explicitly keep the vslidedown narrow?)

Diff Detail

Event Timeline

luke created this revision.May 24 2023, 4:54 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 24 2023, 4:54 PM
luke requested review of this revision.May 24 2023, 4:54 PM
luke edited the summary of this revision. (Show Details)May 24 2023, 4:56 PM
luke added reviewers: khchen, frasercrmck.
craig.topper accepted this revision.EditedMay 24 2023, 7:42 PM

LGTM

SiFive's x280 does use VL to limit the number of ALU cycles. But that only makes a big difference with a small VL and a large LMUL.

This revision is now accepted and ready to land.May 24 2023, 7:42 PM
This revision was automatically updated to reflect the committed changes.