[Openmp-commits] [PATCH] D42686: [libomptarget] Only use CUDA Driver API

[George Rokos via Phabricator via Openmp-commits]

grokos added inline comments.


================
Comment at: libomptarget/plugins/cuda/src/rtl.cpp:691-692
+    CUDA_ERR_STRING(sync_err);
+    assert(sync_err == CUDA_SUCCESS &&
+           "Unable to synchronize target execution!");
     return OFFLOAD_FAIL;
----------------
This assertion is guaranteed to trip - it's inside an `if (sync_err != CUDA_SUCCESS)` - so it doesn't make sense to have it. Besides, if kernel execution fails, the plugin should just return `OFFLOAD_FAIL` to the parent runtime, not terminate the program.


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D42686