This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Zero extend immediate for vget/vset builtins to match vector.insert/extract intrinsics.
ClosedPublic

Authored by craig.topper on Jun 26 2022, 11:37 PM.

Details

Summary

The vector.insert/extract intrinsics require an i64 immediate argument.
This fixes a crash on RV32.

This is an alternative to D128613.

Diff Detail

Event Timeline

craig.topper created this revision.Jun 26 2022, 11:37 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 26 2022, 11:37 PM
craig.topper requested review of this revision.Jun 26 2022, 11:37 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 26 2022, 11:37 PM
craig.topper retitled this revision from [RISCV] Zero immediate for vget/vset builtins to match vector.insert/extract intrinsics. to [RISCV] Zero extend immediate for vget/vset builtins to match vector.insert/extract intrinsics..Jun 27 2022, 1:04 PM
liaolucy accepted this revision.Jun 27 2022, 6:21 PM

LGTM, thanks

This revision is now accepted and ready to land.Jun 27 2022, 6:21 PM