================
Comment at: src/CMakeLists.txt:57
@@ +56,3 @@
+
+string(REPLACE ";" " " link_flags "${link_flags}")
+
----------------
Dan Albert wrote:
> Alexey Samsonov wrote:
> > Do you need to do the same for compile_flags?
> I actually have no idea why this is here. I copied most of this from the 
> libc++ cmake files and tuned as necessary.
> 
> It's done for the compile flags in the main CMakeLists.txt (applied to 
> LIBCXXABI_CXX_*_FLAGS, as these can contain parts specified from the command 
> line using ; as a delimiter). I don't see anything about the ldflags that 
> would need replacement though... I'll remove it.
Well, link_flags is a CMake list type, its elements will be semicolon-separated 
if you add them with list(APPEND link_flags foo).

http://reviews.llvm.org/D4359



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to