<div dir="ltr">Currently, this warning is on by default.  As you said, the results you found look intentional in many cases, so there is a high false positive rate.  For on by default warnings, we expect a high true positive rate and intend for users to not disable the warning.  From my analysis on a separate codebase, I found less than 10% true positive rate out of 200 warnings.  One option might be to move this warning to a subgroup, which would leave it discoverable from either -Wall or -Wparentheses, but not have it on by default.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 28, 2016 at 4:09 AM, Daniel Marjamäki <span dir="ltr"><<a href="mailto:daniel.marjamaki@evidente.se" target="_blank">daniel.marjamaki@evidente.se</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">danielmarjamaki added a comment.<br>
<br>
I updated the patch so it does not warn about 'A * B << C'. It's a simple fix. I have not made careful measurements but I guess that the performance penalty is acceptable.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D24861" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D24861</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>