This is an archive of the discontinued LLVM Phabricator instance.

[8/11][POC][Clang][RISCV] Define tuple type variant of vloxseg2ei32 vluxseg2ei32
ClosedPublic

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

Details

Summary

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

Depends on D147913.

This is the 8th 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 indexed segment load is
not removed, and only signed integer indexed segment load of NF=2,
EEW=32 is defined here.

Diff Detail

Event Timeline

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

Same comment as last review

eopXD updated this revision to Diff 516306.Apr 24 2023, 1:38 AM
eopXD marked an inline comment as done.

Address comments from Craig.

evandro removed a subscriber: evandro.May 17 2023, 3:52 PM
eopXD updated this revision to Diff 523503.May 18 2023, 12:21 PM

Reduce variables declared but only used once.

eopXD updated this revision to Diff 523625.May 18 2023, 5:50 PM

Removed another variable that is too verbose.

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