<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">My bad, it is not r198328 (Clang tree)
but r198344 (LLVM tree), you used the same commit title and I
didn't check the revision when I looked-up the email.<br>
<br>
In particular the "
<meta charset="utf-8">
set_output_directory" addition here:<br>
<br>
<a class="moz-txt-link-freetext" href="http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/modules/AddLLVM.cmake?r1=198344&r2=198343&pathrev=198344">http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/modules/AddLLVM.cmake?r1=198344&r2=198343&pathrev=198344</a><br>
<br>
Mehdi<br>
<br>
<br>
On 1/31/14, 10:41 AM, Jordan Rose wrote:<br>
</div>
<blockquote
cite="mid:C0771620-59D1-4C61-BB03-D71613EF8694@apple.com"
type="cite">
<pre wrap="">I can't see how it would be this commit, since this only applies to targets using add_clang_executable. Takumi?
On Jan 31, 2014, at 1:12, Mehdi Amini <a class="moz-txt-link-rfc2396E" href="mailto:mehdi.amini@silkan.com"><mehdi.amini@silkan.com></a> wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Hi,
It seems this commit breaks out-of-tree build for LLVM passes, as described here <a class="moz-txt-link-freetext" href="http://llvm.org/docs/CMake.html#developing-llvm-pass-out-of-source">http://llvm.org/docs/CMake.html#developing-llvm-pass-out-of-source</a>
I'm not sure if the issue is this commit or r197394 ; as at that time Dmitri Gribenko mentioned that it broke out-of-tree build of Clang.
Mehdi
On 1/2/14, 10:28 AM, Jordan Rose wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Author: jrose
Date: Thu Jan 2 12:28:32 2014
New Revision: 198328
URL: <a class="moz-txt-link-freetext" href="http://llvm.org/viewvc/llvm-project?rev=198328&view=rev">http://llvm.org/viewvc/llvm-project?rev=198328&view=rev</a>
Log:
[CMake] Add missing set_output_directory after Takumi's change in r198205.
In a standalone build, Clang binaries should end up in Clang's build folder,
not LLVM's.
Xcode still has a few issues finding auxiliary tools and libraries in the
build folders. I'll fix those next.
Modified:
cfe/trunk/CMakeLists.txt
Modified: cfe/trunk/CMakeLists.txt
URL: <a class="moz-txt-link-freetext" href="http://llvm.org/viewvc/llvm-project/cfe/trunk/CMakeLists.txt?rev=198328&r1=198327&r2=198328&view=diff">http://llvm.org/viewvc/llvm-project/cfe/trunk/CMakeLists.txt?rev=198328&r1=198327&r2=198328&view=diff</a>
==============================================================================
--- cfe/trunk/CMakeLists.txt (original)
+++ cfe/trunk/CMakeLists.txt Thu Jan 2 12:28:32 2014
@@ -332,6 +332,7 @@ endmacro(add_clang_library)
macro(add_clang_executable name)
add_llvm_executable( ${name} ${ARGN} )
set_target_properties(${name} PROPERTIES FOLDER "Clang executables")
+ set_output_directory(${name} ${CLANG_RUNTIME_OUTPUT_INTDIR} ${CLANG_LIBRARY_OUTPUT_INTDIR})
endmacro(add_clang_executable)
include_directories(BEFORE
_______________________________________________
cfe-commits mailing list
<a class="moz-txt-link-abbreviated" href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a>
<a class="moz-txt-link-freetext" href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a>
</pre>
</blockquote>
<pre wrap="">
</pre>
</blockquote>
<pre wrap="">
</pre>
</blockquote>
<br>
</body>
</html>