<p dir="ltr">Totally makes sense to me, at least on windows</p>
<div class="gmail_quote">10 февр. 2014 г. 18:13 пользователь "Hans Wennborg" <<a href="mailto:hwennborg@google.com">hwennborg@google.com</a>> написал:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This just bit me.<br>
<br>
Do we want to bump the "cmake_minimum_required" version in<br>
compiler-rt's CMakeLists.txt to 2.8.12.1?<br>
<br>
 - Hans<br>
<br>
On Fri, Jan 31, 2014 at 10:31 AM, Timur Iskhodzhanov<br>
<<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>> wrote:<br>
> Ah, thanks for the pointer!<br>
> That explains why I wasn't able to reproduce it on my newer workstation with<br>
> a fresh CMake.<br>
><br>
> I've removed /FS in r200564.<br>
><br>
><br>
> 2014-01-30 Brad King <<a href="mailto:brad.king@kitware.com">brad.king@kitware.com</a>>:<br>
><br>
>> On 1/30/2014 2:53 PM, Timur Iskhodzhanov wrote:<br>
>> > Good idea, done in r200485<br>
>> ><br>
>> ><br>
>> > 2014-01-30 Alexey Samsonov:<br>
>> ><br>
>> >     Does it make sense to add -FS/ to SANITIZER_COMMON_FLAGS instead?<br>
>><br>
>> FYI, CMake 2.8.12.1 was released just after VS 2013 in order<br>
>> to add /FS as part of the generation process:<br>
>><br>
>>  <a href="http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=216afc8a" target="_blank">http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=216afc8a</a><br>
>><br>
>> It should be automatic with that version of CMake and higher.<br>
>><br>
>> -Brad<br>
><br>
><br>
</blockquote></div>