This is an archive of the discontinued LLVM Phabricator instance.

[OpenCL][Docs] Update OpenCL 3.0 implementation status.
ClosedPublic

Authored by Anastasia on Sep 6 2021, 5:55 AM.

Details

Summary

Updated a section of OpenCLSupport page to reflect the latest development in OpenCL 3.0 support for release 13.

This table will be displayed in https://clang.llvm.org/docs/OpenCLSupport.html#opencl-c-3-0-implementation-status.

FYI the current change is going to be added in the main branch as well as the release 13 branch. This therefore reflects the implementation state for the release 13. I am aware that some more patches have been merged and therefore finalized some more features after the release was branched which will then appear in release 14. I am going to follow up with the update including this delta for the main branch separately.

Diff Detail

Event Timeline

Anastasia created this revision.Sep 6 2021, 5:55 AM
Anastasia requested review of this revision.Sep 6 2021, 5:55 AM

@azabaznov and @airlied the main feedback I am looking for is whether I have missed any important patch and whether you agree with the judgement about the completion status.

Note, that due to the release completion schedule and my planned absences it is likely that this needs finalizing by the end of the week. Any small changes can still be added later on should you wish to improve something afterwards up until the release branch is frozen. I just want to make sure that we don't miss adding the main content for the release.

Anastasia added inline comments.Sep 6 2021, 6:18 AM
clang/docs/OpenCLSupport.rst
392

I can see that https://reviews.llvm.org/D107539 is not committed yet... so potentially it would make sense to change to worked on.

azabaznov accepted this revision.Sep 7 2021, 5:21 AM

LGTM, thanks!

This revision is now accepted and ready to land.Sep 7 2021, 5:21 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptSep 10 2021, 4:24 AM
Herald added a subscriber: ldrumm. · View Herald Transcript