[PATCH] D51663: [test-suite, CUDA] Update CUDA test suite cmake files.

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 4 16:32:43 PDT 2018


MatzeB added inline comments.


================
Comment at: cmake/modules/TestSuite.cmake:28-31
+  set(SUFFIX)
+  if(_LTDARGS_SUFFIX)
+    set(SUFFIX ${_LTDARGS_SUFFIX})
+  endif()
----------------
As the default is undefined/empty suffix anyway, can't you use `${_LTDARGS_SUFFIX}` directly below? Or go with a simple `set(SUFFIX ${_LTDAGS_SUFFIX}` without the if here?



Repository:
  rT test-suite

https://reviews.llvm.org/D51663





More information about the llvm-commits mailing list