This is an archive of the discontinued LLVM Phabricator instance.

Add explicit index type for llvm.vector.extract
AbandonedPublic

Authored by liaolucy on Jun 26 2022, 8:31 AM.

Details

Summary

Without this patch, we get: llvm.experimental.vector.extract.nxv.nxv

Whereas with the patch we get:
RV32, the intrinsic name is: llvm.experimental.vector.extract.nxv.nxv.i32
RV64, the intrinsic name is: llvm.experimental.vector.extract.nxv.nxv.i64

Diff Detail

Event Timeline

liaolucy created this revision.Jun 26 2022, 8:31 AM
liaolucy requested review of this revision.Jun 26 2022, 8:31 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJun 26 2022, 8:31 AM

The patch : fix crash for rv32 vget intrinsics

Can we just add the missing zext to the RISC-V vget implementation?

liaolucy abandoned this revision.Jun 27 2022, 6:31 PM