The notes cover functionality in clang added by commits from 16 July to 27th Jan.
Diff Detail
Event Timeline
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. |
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. |
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. |
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. |
clang/docs/ReleaseNotes.rst | ||
---|---|---|
214 | No preference, but we could even mention both terms for completeness perhaps? |
clang/docs/ReleaseNotes.rst | ||
---|---|---|
214 | Do you mean something like: - Allow nested pointers (e.g. pointer-to-pointer) kernel arguments beyond OpenCL 1.2. |
clang/docs/ReleaseNotes.rst | ||
---|---|---|
214 | Looks good to me. |
- 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?
Committed to the release branch in https://github.com/llvm/llvm-project/commit/00441b8f4e5b7daa39ac6cbeb45ebfe54662b08d