[Openmp-commits] [PATCH] D45530: [OpenMP] Make bc file compilation sensitive to LIBOMPTARGET_NVPTX_DEBUG flag

[Guansong Zhang via Phabricator via Openmp-commits]

guansong added inline comments.


================
Comment at: libomptarget/deviceRTLs/nvptx/CMakeLists.txt:146
+      # Activate RTL message dumps if requested by the user.
+      set(LIBOMPTARGET_BC_DEBUG FALSE CACHE BOOL
+        "Activate BC code device RTL debug messages.")
----------------
Hahnfeld wrote:
> Hmm, why does `Sync` mean a new variable? If that's what you want, it should start with `LIBOMPTARGET_NVPTX` and be properly documented
Thanks. I put a typo there, it should be 

set(BC_DEBUG FALSE CACHE BOOL

As I used in the next few lines. BC_DEBUG is an internal var. The behavior to sync is the Cmake command line def LIBOMPTARGET_NVPTX_DEBUG


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D45530