This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Legalize fixed length (insert_subvector undef, X, 0) to a scalable insert.
ClosedPublic

Authored by craig.topper on Jan 14 2022, 6:03 PM.

Details

Summary

We were considering this legal, but later the undef would become an all
zeros vector. This would cause us to need to re-legalize the insert later
into a vslideup with zero vector.

This patch catches the case and directly legalizes it to a scalable
insert.

Diff Detail

Event Timeline

craig.topper created this revision.Jan 14 2022, 6:03 PM
craig.topper requested review of this revision.Jan 14 2022, 6:03 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 14 2022, 6:03 PM
Herald added a subscriber: MaskRay. · View Herald Transcript
frasercrmck accepted this revision.Jan 17 2022, 3:50 AM

LGTM, cheers

This revision is now accepted and ready to land.Jan 17 2022, 3:50 AM
This revision was landed with ongoing or failed builds.Jan 17 2022, 2:40 PM
This revision was automatically updated to reflect the committed changes.