<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 24, 2014 at 12:48 PM, Yaron Keren <span dir="ltr"><<a href="mailto:yaron.keren@gmail.com" target="_blank">yaron.keren@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":108" class="a3s" style="overflow:hidden">+#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 warns in<br>
+// Release modes but not in Debug modes. The real reason is optimization flags.<br>
+// With /Od in Debug modes the warning is not issued whereas with /O1 it is.<br>
+// I could not find any documentation to this effect, it is reproducable:<br>
+// Try compiling <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)</div></blockquote></div><br>Is this warning providing any value at all? I wonder if we should just disable it in cmake as low value. The fundamental error is well covered by Clang's existing warnings.</div>
</div>