This is an archive of the discontinued LLVM Phabricator instance.

[Clang][RISCV] Implement vlsseg.
ClosedPublic

Authored by HsiangKai on Jun 7 2021, 2:56 AM.

Diff Detail

Event Timeline

HsiangKai created this revision.Jun 7 2021, 2:56 AM
HsiangKai requested review of this revision.Jun 7 2021, 2:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 7 2021, 2:56 AM
HsiangKai updated this revision to Diff 357716.Jul 10 2021, 7:16 AM
craig.topper added inline comments.Jul 15 2021, 10:47 PM
clang/include/clang/Basic/riscv_vector.td
955

I don't think this alignment is correct. A vint16mf4_t creates an alloca with align of 2 and vint8mf4_t creates an alloca with an align of 1. So I think the store here needs to match the alignment you would get for the type we're storing. This is an issue in the earlier vlseg patch as well.

HsiangKai updated this revision to Diff 359703.Jul 19 2021, 2:19 AM

Correct the alignment of store.

Remove RV32 test cases.

This revision is now accepted and ready to land.Jul 21 2021, 10:02 AM
This revision was landed with ongoing or failed builds.Jul 21 2021, 6:25 PM
This revision was automatically updated to reflect the committed changes.
clang/include/clang/Basic/riscv_vector.td