[PATCH] D16242: [test-suite] Move LDFLAGS to after objects

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 15 13:36:32 PST 2016


MatzeB accepted this revision.
MatzeB added a comment.
This revision is now accepted and ready to land.

It surprised me that target_link_libraries can be used for arbitrary flags, but the cmake docu says so so this is fine.
I wonder if you can just replace the two `append_ldflags(${source_exename} LDFLAGS)` calls with `target_link_libraries(${source_exename} ${LDFLAGS})` and remove the whole append_ldflags macro?


http://reviews.llvm.org/D16242





More information about the llvm-commits mailing list