[Openmp-commits] [PATCH] D141933: [Libomptarget] Replace Nvidia arch lookup with 'nvptx-arch'

Joseph Huber via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Jan 17 09:02:47 PST 2023


jhuber6 added inline comments.


================
Comment at: openmp/libomptarget/test/lit.cfg:88
   try:
-    cuda_arch = int(config.cuda_test_arch)
+      cuda_arch = int(config.cuda_test_arch[:3])
     if cuda_arch < 70:
----------------
tianshilei1992 wrote:
> why this?
The indentation was an accident. The slice is because the old version returned just `70` while the new one would output `sm_70`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141933



More information about the Openmp-commits mailing list