[PATCH] D58504: [OpenCL][8.0.0 Release] Notes for OpenCL

Anastasia Stulova via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 21 11:24:28 PST 2019


Anastasia marked an inline comment as done.
Anastasia added inline comments.


================
Comment at: docs/ReleaseNotes.rst:228
 
-OpenCL C Language Changes in Clang
+OpenCL Language Changes in Clang
 ----------------------------------
----------------
AlexeySotkin wrote:
> AlexeySotkin wrote:
> > Why the "C" is removed ?
> Should we call the section like: "OpenCL Support in Clang"?
I would like to leave "Changes" in as it described the difference in this release. And I think "Language" should stay to be more precise that this doesn't cover libraries or runtime.

So I was think of either:
`OpenCL Kernel Language Change in Clang`
 or
`'OpenCL C/C++ Language Changes in Clang'`

I am a bit worried about the last one since it might link to the OpenCL C++ spec... even though it was used in the past release notes... but now that the direction has been changed I am not sure. Any opinion?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58504/new/

https://reviews.llvm.org/D58504





More information about the cfe-commits mailing list