<div dir="ltr">On 14 October 2013 15:25, Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@britannica.bec.de" target="_blank">joerg@britannica.bec.de</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">On Mon, Oct 14, 2013 at 02:45:22PM -0700, Nick Lewycky wrote:<br>


> As a practical matter, I'm adding these so that I can continue to build<br>
> software that used to build. They're marked clang_ignored_[fm]_Group so<br>
> that we can find which flags aren't implemented.<br>
<br>
</div>I have a certain issue with adding misguided flags that might have made<br>
seen two decades ago and got C&P around.</blockquote><div><br></div><div>GCC compatible driver.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div class="im">
> I'm not sure what you're saying -- are you arguing that implementing<br>
> -minline-all-stringops is dangerous, or that silently ignoring<br>
> -minline-all-stringops is dangerous?<br>
<br>
</div>Silently ignoring it is wrong. The behavior it is supposed to provide is<br>
not available.<br></blockquote><div><br></div><div>I'm restoring behaviour after r191328/191394 -- it makes more sense if you think of this patch as being a part of those patches. We're moving from silently accepting all unknown flags to a whitelist of known gcc flags which we'll silently ignore. This keeps us closer to honest by documenting in the .td file which flags we know we don't implement. It's a step in the right direction.</div>

<div><br></div><div>The alternative is to revert r191328 and r191394, and go back to silently allowing all flags. Breaking builds that used to work really isn't acceptable. If I had known in advance how long it would take to find all the unsupported flags in my build, I would have reverted.</div>

<div><br></div><div>Longer term, we should implement the ones that are sensible and warn on the ones that aren't.</div><div><br></div><div>Nick</div></div></div></div>