This warning group (-Wshift-op-parentheses) is already a subgroup of -Wparentheses. The suggestion of moving to a sub group was to create a new subgroup, either as a child of -Wshift-op-parentheses or -Wparentheses, so that this new warning can be turned on and off independently and allow it to be off by default.
On Mon, Oct 3, 2016 at 3:26 AM, Daniel Marjamäki < [email protected]> wrote: > Hello! > > Moving it to a subgroup such as -Wparentheses is fine for me. I will look > at that when I have time. Do you think this warning has an acceptable > output then? > > Best regards, > Daniel Marjamäki > > ............................................................ > ...................................................... > Daniel Marjamäki Senior Engineer > Evidente ES East AB Warfvinges väg 34 SE-112 51 Stockholm Sweden > > Mobile: +46 (0)709 12 42 62 > E-mail: [email protected] > > www.evidente.se > > ________________________________________ > Från: Richard Trieu [[email protected]] > Skickat: den 1 oktober 2016 00:41 > Till: [email protected] > Kopia: Daniel Marjamäki; David Blaikie; [email protected]; > [email protected]; cfe-commits > Ämne: Re: [PATCH] D24861: [Sema] extend Wshift-op-parentheses so it warns > for multiplicative operators > > 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. > > On Wed, Sep 28, 2016 at 4:09 AM, Daniel Marjamäki < > [email protected]<mailto:[email protected]>> wrote: > danielmarjamaki added a comment. > > 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. > > > https://reviews.llvm.org/D24861 > > > > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
