I'll have a peek when I get to a computer (on vaca...)<span></span><br><br>On Thursday, July 31, 2014, Chandler Carruth <<a href="mailto:chandlerc@gmail.com">chandlerc@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Minor comment tweak. Otherwise this looks good now. Thanks.<br>
<br>
I guess Greg wants you to wait for Todd's LGTM? :: shruug :: I know nothing about LLDB, just cmake. =]<br>
<br>
================<br>
Comment at: test/CMakeLists.txt:18-19<br>
@@ -17,14 +17,4 @@<br>
<br>
-# Users can override LLDB_TEST_ARGS to modify the way LLDB tests are run. See help below.<br>
-set(LLDB_TEST_ARGS<br>
-  -C<br>
-  ${LLDB_TEST_COMPILER}<br>
-  CACHE STRING "Specify compiler(s) and architecture(s) with which run LLDB tests. For example: '-C gcc -C clang -A i386 -A x86_64'"<br>
-  )<br>
-string(REPLACE " " ";" LLDB_TEST_ARGS ${LLDB_TEST_ARGS})<br>
-<br>
-set(LLDB_TRACE_DIR "${CMAKE_BINARY_DIR}/lldb-test-traces"<br>
-  CACHE STRING "Set directory to output LLDB test traces (for tests that do not pass.)"<br>
-  )<br>
-<br>
-set(LLDB_COMMON_TEST_ARGS<br>
+# The default architecture with which to compile test executables is the architecture that LLDB<br>
+# itself was built with.<br>
+string(TOLOWER "${LLVM_TARGET_ARCH}" LLDB_DEFAULT_TEST_ARCH)<br>
----------------<br>
This comment isn't quite accurate... Maybe:<br>
<br>
"The default architecture with which to compile test executables is the default LLVM target architecture, which itself defaults to the host architecture."<br>
<br>
<a href="http://reviews.llvm.org/D4730" target="_blank">http://reviews.llvm.org/D4730</a><br>
<br>
<br>
<br>
_______________________________________________<br>
lldb-commits mailing list<br>
<a href="javascript:;" onclick="_e(event, 'cvml', 'lldb-commits@cs.uiuc.edu')">lldb-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
</blockquote><br><br>-- <br><div dir="ltr"><table cellspacing="0" cellpadding="0" style="color:rgb(136,136,136);font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small">
<td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Todd Fiala |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tfiala@google.com" style="color:rgb(17,85,204)" target="_blank"><span style="background-color:rgb(255,255,204);color:rgb(34,34,34);background-repeat:initial initial">tfiala@google.com</span></a> |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font color="#1155cc"> <a>650-943-3180</a></font></td></tr></tbody></table><br></div><br>