<div dir="ltr">I agree that this isn't the perfect approach, but I don't think we want to take the performance penalty "-fno-strict-aliasing" will cause (we are using LLDB like this for a long time without any known issue caused by strict aliasing violations). The best solution would be to fix the strict aliasing violations or validate that they are actually false positives (and work them around) but it haven't happened in the last ~1 year and I don't expect it to happen in the near future either. My primary reason to disable these warnings is to make the other (less noisy) gcc warnings more visible so we can detect possible real problems from them. Having this warning enabled at the current state have almost zero value as AFAIK nobody cares about then and nobody would notice if a new one appear because of the general noise level.<div><br></div><div>Tamas</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 23, 2015 at 1:24 PM Joerg Sonnenberger <<a href="mailto:joerg@britannica.bec.de">joerg@britannica.bec.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Oct 23, 2015 at 10:34:54AM -0000, Tamas Berghammer via lldb-commits wrote:<br>
> Author: tberghammer<br>
> Date: Fri Oct 23 05:34:53 2015<br>
> New Revision: 251107<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=251107&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=251107&view=rev</a><br>
> Log:<br>
> Disable the strict-aliasing warnings produced by gcc<br>
><br>
> GCC produce a lot of strict-aliasing warning for the LLDB codebase<br>
> what makes reading the compile output very difficult. This change<br>
> disable these warnings to reduce the noise as we already ignore them.<br>
><br>
> We should consider re-enabling the warning if we fix all (or most)<br>
> strict-aliasing violation first.<br>
<br>
This feels very wrong. If you want to disable the warnings, at least<br>
also disable the codegen dependency with -fno-strict-aliasing.<br>
<br>
Joerg<br>
</blockquote></div>