[Openmp-commits] [PATCH] D95584: [OpenMP][NFC] Added release note for new `deviceRTLs` and hidden helper task

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

LG, some suggestions for the final version.



================
Comment at: openmp/docs/ReleaseNotes.rst:66
+  an OpenMP program. If the library is not found in ``LIBRARY_PATH``, an error
+  will be raised. User can also specify it via ``--libomptarget-nvptx-bc-path=``.
----------------
"implemented (via|with|using) ... intrinsics"
non-CUDA system -> CUDA-free system (maybe)
Say that if CUDA is specified or found, the behavior did not change and CUDA is used.

Make the static NVPTX runtime replacement a new bullet point please.

"A bitcode device library is"
"If the library is not found in the default path or any of the paths defined by the ``LIBRARY_PATH`` environment variable, an error will be raised."
"User can also specify it via" -> "Users can specify the path to the bitcode device library via"



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95584