This is an archive of the discontinued LLVM Phabricator instance.

[6/11][POC][Clang][RISCV] Define tuple type variant of vlsseg2e32
ClosedPublic

Authored by eopXD on Apr 10 2023, 12:41 AM.

Details

Summary

For the cover letter of this patch-set, please checkout D146872.

Depends on D147911.

This is the 6th patch of the patch-set.

This patch is a proof-of-concept and will be extended to full coverage
in the future. Currently, the old non-tuple strided segment load is not
removed, and only signed integer strided segment load of NF=2, EEW=32
is defined here.

Diff Detail

Event Timeline

eopXD created this revision.Apr 10 2023, 12:41 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 10 2023, 12:41 AM
eopXD requested review of this revision.Apr 10 2023, 12:41 AM
eopXD retitled this revision from [6/N][POC][Clang] Define tuple type variant of vlsseg2e32 to [6/11][POC][Clang][RISCV] Define tuple type variant of vlsseg2e32.Apr 10 2023, 12:53 AM
eopXD edited the summary of this revision. (Show Details)
craig.topper added inline comments.Apr 18 2023, 4:48 PM
clang/include/clang/Basic/riscv_vector.td
1677

similar comment to what I left on the last review

eopXD updated this revision to Diff 516301.Apr 24 2023, 1:11 AM

Address comment from Craig.

eopXD marked an inline comment as done.Apr 24 2023, 1:11 AM
evandro removed a subscriber: evandro.May 17 2023, 3:54 PM
craig.topper added inline comments.May 18 2023, 10:42 AM
clang/include/clang/Basic/riscv_vector.td
1678

Same comment as previous patch

eopXD updated this revision to Diff 523485.May 18 2023, 11:48 AM
eopXD marked an inline comment as done.

Address comments from Craig.

eopXD updated this revision to Diff 523622.May 18 2023, 5:44 PM

Removed another variable that is too verbose.

This revision is now accepted and ready to land.May 18 2023, 6:25 PM
This revision was landed with ongoing or failed builds.May 22 2023, 4:09 AM
This revision was automatically updated to reflect the committed changes.