[PATCH] D19424: [test-suite] Propagate LDFLAGS variable to executable's LINK_FLAGS property

Artem Belevich via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 22 11:35:50 PDT 2016


tra created this revision.
tra added reviewers: MatzeB, jmolloy.
tra added a subscriber: llvm-commits.

... which is better match for LDFLAGS which may have options other than -l that should appear on linker command line earlier.


http://reviews.llvm.org/D19424

Files:
  cmake/modules/SingleMultiSource.cmake

Index: cmake/modules/SingleMultiSource.cmake
===================================================================
--- cmake/modules/SingleMultiSource.cmake
+++ cmake/modules/SingleMultiSource.cmake
@@ -79,7 +79,7 @@
 endmacro()
 
 macro(append_link_flags target)
-  append_target_flags(LINK_LIBRARIES ${target} ${ARGN})
+  append_target_flags(LINK_FLAGS ${target} ${ARGN})
 endmacro()
 
 # Traditionally CMakeLists.txt files would set RUN_OPTIONS, STDIN_FILENAME,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19424.54685.patch
Type: text/x-patch
Size: 463 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160422/28c0071c/attachment.bin>


More information about the llvm-commits mailing list