<div dir="rtl"><div dir="ltr">Sure, fine by me to disable it completely and remove the code from both places. </div><div dir="ltr">I'll do this tomorrow,</div><div dir="ltr"><br></div><div dir="ltr">Yaron</div><div dir="ltr">

<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div dir="ltr">2014-03-24 23:09 GMT+02:00 Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span>:</div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Mon, Mar 24, 2014 at 5:05 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>


><br>
> On Mon, Mar 24, 2014 at 12:48 PM, Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>> wrote:<br>
>><br>
>> +#if defined(_MSC_VER)<br>
>> +// In Release modes, Visual Studio complains that the Operator destructor<br>
>> +// never returns, which is true by design.<br>
>> +// This does *not* depend on llvm_unreachable being dependent on NDEBUG:<br>
>> +// even if llvm_unreachable is replaced by __assume(false), VC still<br>
>> warns in<br>
>> +// Release modes but not in Debug modes. The real reason is optimization<br>
>> flags.<br>
>> +// With /Od in Debug modes the warning is not issued whereas with /O1 it<br>
>> is.<br>
>> +// I could not find any documentation to this effect, it is reproducable:<br>
>> +// Try compiling<br>
>> <a href="http://msdn.microsoft.com/en-us/library/khwfyc5d(v=vs.90).aspx" target="_blank">http://msdn.microsoft.com/en-us/library/khwfyc5d(v=vs.90).aspx</a><br>
>> +// with /O1 and then with /Od.<br>
>> +// Anyhow, solution is same as lib/Support/Process.cpp:~self_process().<br>
>> +<br>
>> +#pragma warning(push)<br>
>> +#pragma warning(disable:4722)<br>
><br>
><br>
> Is this warning providing any value at all? I wonder if we should just<br>
> disable it in cmake as low value. The fundamental error is well covered by<br>
> Clang's existing warnings.<br>
<br>
</div></div>I'd be amenable to disabling this one in CMake. It doesn't seem like<br>
it'd be a high-value diagnostic compared to the amount of pain working<br>
around it.<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br></div>