<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Oct 1, 2016 at 9:34 AM, Joerg Sonnenberger via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Sep 30, 2016 at 03:41:53PM -0700, Richard Trieu via cfe-commits wrote:<br>
> Currently, this warning is on by default.  As you said, the results you<br>
> found look intentional in many cases, so there is a high false positive<br>
> rate.  For on by default warnings, we expect a high true positive rate and<br>
> intend for users to not disable the warning.  From my analysis on a<br>
> separate codebase, I found less than 10% true positive rate out of 200<br>
> warnings.  One option might be to move this warning to a subgroup, which<br>
> would leave it discoverable from either -Wall or -Wparentheses, but not<br>
> have it on by default.<br>
<br>
</span>We are now only talking about the right shift version, are we? That<br>
seems to me to be much less intrusive, but should belong into the same<br>
subgroup as the add checks.<br>
<br>
Joerg<br></blockquote><div><br></div><div>I used the updated patch for the warning, which has both left and right shifts, with an exception for "A * B << C".  The number of right shift versus left shift warnings is similar with similar true positive rate. </div></div></div></div>