[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