<p dir="ltr">Re: -g<br>
1) clang crashes on windows with -g [fixable]<br>
2) we had a chat with Kostya and decided it's the right thing to test ASan with just glto to be sure it gives us enough DI</p>
<div class="gmail_quote">24 мая 2014 г. 2:07 пользователь "Alexey Samsonov" <<a href="mailto:vonosmas@gmail.com">vonosmas@gmail.com</a>> написал:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
================<br>
Comment at: cmake/Modules/AddCompilerRT.cmake:151<br>
@@ -151,3 +150,3 @@<br>
     COMMAND ${COMPILER_RT_TEST_COMPILER} ${TEST_OBJECTS}<br>
-            ${COMPILER_RT_TEST_COMPILER_EXE}"${output_bin}"<br>
+            -o"${output_bin}"<br>
             ${TEST_LINK_FLAGS}<br>
----------------<br>
Restore a space here.<br>
<br>
================<br>
Comment at: cmake/Modules/CompilerRTCompile.cmake:24<br>
@@ +23,3 @@<br>
+  if(MSVC)<br>
+    string(REPLACE ";/" ";-" global_flags "${global_flags}")<br>
+  endif()<br>
----------------<br>
what if the first element of global_flags starts with / ?<br>
<br>
================<br>
Comment at: cmake/Modules/CompilerRTCompile.cmake:33<br>
@@ -27,3 +32,3 @@<br>
     COMMAND ${COMPILER_RT_TEST_COMPILER} ${compile_flags} -c<br>
-            ${COMPILER_RT_TEST_COMPILER_OBJ}"${object_file}"<br>
+            -o"${object_file}"<br>
             ${source_rpath}<br>
----------------<br>
and here.<br>
<br>
================<br>
Comment at: lib/asan/tests/CMakeLists.txt:29<br>
@@ -28,1 +28,3 @@<br>
   -I${COMPILER_RT_SOURCE_DIR}/lib/sanitizer_common/tests<br>
+  -DGTEST_HAS_RTTI=0<br>
+  -fno-rtti<br>
----------------<br>
Consider adding this (and, probably, -DGTEST_HAS_SEH=0 on Windows) to COMPILER_RT_GTEST_CFLAGS definition instead.<br>
<br>
================<br>
Comment at: lib/asan/tests/CMakeLists.txt:31<br>
@@ +30,3 @@<br>
+  -fno-rtti<br>
+  -gline-tables-only<br>
+  -O2<br>
----------------<br>
Any particular reason to switch to -gline-tables-only from -g here?<br>
<br>
<a href="http://reviews.llvm.org/D3893" target="_blank">http://reviews.llvm.org/D3893</a><br>
<br>
<br>
</blockquote></div>