[Openmp-commits] [PATCH] D127505: [Libomptarget] Add checks for CUDA subarchitecture using new info

[Shilei Tian via Phabricator via Openmp-commits]

tianshilei1992 added inline comments.


================
Comment at: openmp/libomptarget/plugins/cuda/src/rtl.cpp:1518
+    if (ArchStr == image->Info.Arch)
+      return true;
+  }
----------------
I think here we should check if all devices are compatible instead of one.
```
  if (ArchStr != image->Info.Arch)
    return true;
}

return true.
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127505