This is an archive of the discontinued LLVM Phabricator instance.

[OpenCL][Docs] Release 12.0 notes
ClosedPublic

Authored by Anastasia on Mar 5 2021, 1:15 PM.

Details

Summary

The notes cover functionality in clang added by commits from 16 July to 27th Jan.

Diff Detail

Event Timeline

Anastasia created this revision.Mar 5 2021, 1:15 PM
Anastasia requested review of this revision.Mar 5 2021, 1:15 PM
svenvh added inline comments.Mar 9 2021, 3:51 AM
clang/docs/ReleaseNotes.rst
208
211
214

Perhaps one more point to mention:

- Allow pointer-to-pointer kernel arguments beyond OpenCL 1.2.

(related commit is 523775f96742 ("[OpenCL] Allow pointer-to-pointer kernel args beyond CL 1.2", 2020-12-01)).

222

Nitpick: you're mixing past and present tense ("added" vs "fix"). It would be nicer for consistency to use the same tense everywhere.

Anastasia updated this revision to Diff 329617.Mar 10 2021, 4:53 AM

Added corrections from Sven.

Anastasia marked 3 inline comments as done.Mar 10 2021, 4:54 AM
Anastasia added inline comments.
clang/docs/ReleaseNotes.rst
214

I have covered it in this but it is combined with another diagnostic:

- Improved diagnostics for nested pointers and unevaluated ``vec_step`` expression.

But if you think it is better I can change as you have suggested.

svenvh added inline comments.Mar 10 2021, 6:36 AM
clang/docs/ReleaseNotes.rst
214

Ah, that's probably why I missed it, as I was assuming "improved diagnostics" means a nicer error message, not that Clang now accepts something that it rejected before. So in my opinion it deserves a point on its own.

Anastasia added inline comments.Mar 10 2021, 6:41 AM
clang/docs/ReleaseNotes.rst
214

Ok, do you think we should use nested pointers instead of pointer-to-pointer because your change seems to work with any number of pointers. Although I appreciate that the latter one is a spec terminology.

svenvh added inline comments.Mar 10 2021, 6:44 AM
clang/docs/ReleaseNotes.rst
214

No preference, but we could even mention both terms for completeness perhaps?

Anastasia added inline comments.Mar 10 2021, 7:49 AM
clang/docs/ReleaseNotes.rst
214

Do you mean something like:

- Allow nested pointers (e.g. pointer-to-pointer) kernel arguments beyond OpenCL 1.2.
svenvh accepted this revision.Mar 10 2021, 8:36 AM
svenvh added inline comments.
clang/docs/ReleaseNotes.rst
214

Looks good to me.

This revision is now accepted and ready to land.Mar 10 2021, 8:36 AM
Anastasia updated this revision to Diff 329944.Mar 11 2021, 5:50 AM
  • Added nested pointer item
  • Elaborated list of removed extensions with no kernel language changes.

Sorry for the last-minute change, I have decided to add this too:

+- Removed extensions without kernel language changes:
+  ``cl_khr_select_fprounding_mode``, ``cl_khr_gl_sharing``, ``cl_khr_icd``,
+  ``cl_khr_gl_event``, ``cl_khr_d3d10_sharing``, ``cl_khr_context_abort``,
+  ``cl_khr_d3d11_sharing``, ``cl_khr_dx9_media_sharing``,
+  ``cl_khr_image2d_from_buffer``, ``cl_khr_initialize_memory``,
+  ``cl_khr_gl_depth_images``, ``cl_khr_spir``, ``cl_khr_egl_event``,
+  ``cl_khr_egl_image``, ``cl_khr_terminate_context``.

@tstellar is it ok if I commit this to the release branch or do you prefer to do it yourself?

svenvh accepted this revision.Mar 11 2021, 5:58 AM

Sorry for the last-minute change, I have decided to add this too:

+- Removed extensions without kernel language changes:
+  ``cl_khr_select_fprounding_mode``, ``cl_khr_gl_sharing``, ``cl_khr_icd``,
+  ``cl_khr_gl_event``, ``cl_khr_d3d10_sharing``, ``cl_khr_context_abort``,
+  ``cl_khr_d3d11_sharing``, ``cl_khr_dx9_media_sharing``,
+  ``cl_khr_image2d_from_buffer``, ``cl_khr_initialize_memory``,
+  ``cl_khr_gl_depth_images``, ``cl_khr_spir``, ``cl_khr_egl_event``,
+  ``cl_khr_egl_image``, ``cl_khr_terminate_context``.

Makes sense, LGTM!