[Openmp-commits] [PATCH] D41724: [OpenMP][libomptarget] Enable the compilation of multiple bc libraries for runtime inlining

[Jonas Hahnfeld via Phabricator via Openmp-commits]

Hahnfeld added inline comments.


================
Comment at: README.rst:283-286
  **LIBOMPTARGET_NVPTX_COMPUTE_CAPABILITY** = ``35``
   CUDA compute capability that should be supported by the NVPTX device RTL. E.g.
   for compute capability 6.0, the option "60" should be used. Compute capability
   3.5 is the minimum required.
----------------
grokos wrote:
> Maybe add a comment here that the single-capability option is deprecated and the user should use the next option to define compute capabilities (even if they want to define only one)?
I think we should just remove it: There was no (upstream) release with this variable and the only code checking for it is there for compatibility.


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D41724