[PATCH] D20842: [test-suite] Simplified test executable name generation.

Artem Belevich via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 1 13:17:57 PDT 2016


tra updated this revision to Diff 70062.
tra marked an inline comment as done.
tra added a comment.

- prefix local variables in macros.
- check whether we've created a target before using it.
- reset TARGET_VAR variable in llvm_test_executable, so end-user does not have to.
- return target info from llvm_multisource. It was previously clobbered.

@MatzeB - I'd appreciate if you could take another look.


https://reviews.llvm.org/D20842

Files:
  SingleSource/Benchmarks/Shootout-C++/CMakeLists.txt
  SingleSource/Benchmarks/Shootout/CMakeLists.txt
  SingleSource/Regression/C++/CMakeLists.txt
  SingleSource/Regression/C/CMakeLists.txt
  SingleSource/UnitTests/Vector/CMakeLists.txt
  SingleSource/UnitTests/Vector/NEON/CMakeLists.txt
  cmake/modules/SingleMultiSource.cmake

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D20842.70062.patch
Type: text/x-patch
Size: 8849 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160901/b1392025/attachment.bin>


More information about the llvm-commits mailing list