This is an archive of the discontinued LLVM Phabricator instance.

[Doc] Proposal for vector predication
ClosedPublic

Authored by simoll on Feb 3 2020, 6:28 AM.

Details

Summary

Proposal and roadmap towards vector predication in LLVM.
This patch documents that
a) It is recognized that current LLVM is ill-equipped for vector predication.
b) The community is working on a solution.
c) A concrete prototype exists in the VP extension (D57504).

Diff Detail

Event Timeline

simoll created this revision.Feb 3 2020, 6:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 3 2020, 6:28 AM
simoll added a comment.Feb 3 2020, 6:34 AM

I probably did not add everybody who should be reviewing or at least be aware of this. Feel free to add folks to make sure we get wide community agreement going forward.

Unit tests: pass. 62414 tests passed, 0 failed and 839 were skipped.

clang-tidy: pass.

clang-format: pass.

Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml

Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.

I feel like the bar should be fairly low for committing this. The proposal itself will notify those who are interested (agree or disagree) where they can join the discussion.

Many lines are over 80 characters (some people read these in raw rst format). Other than that, it looks good to me.

simoll updated this revision to Diff 242821.Feb 6 2020, 12:53 AM

Formatted to 80 characters per line max

I feel like the bar should be fairly low for committing this. The proposal itself will notify those who are interested (agree or disagree) where they can join the discussion.

I guess you are right. This is a proposal not a decision after all.

Unit tests: unknown.

clang-tidy: unknown.

clang-format: unknown.

Build artifacts: console-log.txt

Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.

This revision is now accepted and ready to land.Feb 7 2020, 1:54 PM
This revision was automatically updated to reflect the committed changes.
rkruppe removed a reviewer: rkruppe.Apr 4 2021, 4:08 AM