<div dir="ltr">Do you think it's OK to add <span style="font-family:arial,sans-serif;font-size:13px">COMPILER_RT_TEST_CFLAGS which might be empty on Linux if you prefer</span><span style="font-family:arial,sans-serif;font-size:13px"> (or maybe -fno-rtti -g*?)</span><span style="font-family:arial,sans-serif;font-size:13px"> but non-empty on Windows?</span></div>

<div class="gmail_extra"><br><br><div class="gmail_quote">2014-05-28 23:13 GMT+04:00 Alexey Samsonov <span dir="ltr"><<a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Wed, May 28, 2014 at 2:06 AM, Timur Iskhodzhanov <span dir="ltr"><<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2014-05-28 12:49 GMT+04:00 Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>>:<br>



<div>> I should probably move the stuff we conditionally append to SANITIZER_TEST_CFLAGS_COMMON to COMPILER_RT_GTEST_CFLAGS instead.  Will do that in a separate commit.<br>
<br>
</div>On the second thought, the _GTEST_ part of the macro name doesn't fit.</blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Any objections if I define COMPILER_RT_TEST_CFLAGS in<br>
cmake/Modules/AddCompilerRT.cmake ?<br>
It should probably include other common stuff like -O2 and<br>
-g[line-tables-only] ?<br></blockquote><div><br></div></div></div><div>I'd prefer to keep GTest-relevant defines in _GTEST_ a separate variable as it's done at the moment. Not sure adding one more variable for unittest common compile flags would be good,</div>


<div>e.g. I don't see why we should build all the tests with -O2 (one moment we may decide to test other opt levels). Listing these flags every time we add a specific unittest binary is not that bad.</div><span class="HOEnZb"><font color="#888888"><div>

<br></div>
</font></span></div><span class="HOEnZb"><font color="#888888"><div><br></div>-- <br><div dir="ltr">Alexey Samsonov<br><a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a></div>
</font></span></div></div>
</blockquote></div><br></div>