This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Use vslide1down for i64 insertelt on RV32.
ClosedPublic

Authored by craig.topper on Oct 25 2022, 10:04 PM.

Details

Summary

Instead of using vslide1up, use vslide1down and build the other
direction. This avoids the overlap constraint early clobber of
vslide1up.

Diff Detail

Event Timeline

craig.topper created this revision.Oct 25 2022, 10:04 PM
craig.topper requested review of this revision.Oct 25 2022, 10:04 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 25 2022, 10:04 PM

Update comment

reames accepted this revision.Oct 26 2022, 8:50 AM

Good idea. LGTM

Should we be preferring this when building small build_vectors too?

This revision is now accepted and ready to land.Oct 26 2022, 8:50 AM
This revision was landed with ongoing or failed builds.Oct 26 2022, 9:43 AM
This revision was automatically updated to reflect the committed changes.