<div dir="ltr">r322901.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 18, 2018 at 11:03 AM, Roman Lebedev <span dir="ltr"><<a href="mailto:lebedev.ri@gmail.com" target="_blank">lebedev.ri@gmail.com</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 Thu, Jan 18, 2018 at 6:59 PM, Nico Weber <<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>> wrote:<br>
> Marshall,<br>
<br>
> I'll remove the warning from -Wextra today<br>
</span>Thank you!<br>
<div class="HOEnZb"><div class="h5"><br>
> I just need to find<br>
> some time to collect links to all the emails asking for that from various<br>
> people for the commit message.<br>
><br>
> On Thu, Jan 18, 2018 at 9:26 AM, Roman Lebedev <<a href="mailto:lebedev.ri@gmail.com">lebedev.ri@gmail.com</a>> wrote:<br>
>><br>
>> On Wed, Jan 17, 2018 at 6:25 PM, Nico Weber <<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>> wrote:<br>
>> > Roman, do you want to remove the warning from -Wextra, or do you want me<br>
>> > to<br>
>> > do it?<br>
>> To reiterate, i have committed exactly the code that was in the<br>
>> differential,<br>
>> and was reviewed.<br>
>> If there is a desire to change that, then by all means, do it,<br>
>> feel free to remove it from -Wextra, there won't be any reverts from me.<br>
>><br>
>> > On Tue, Jan 16, 2018 at 12:59 PM, Nico Weber <<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>><br>
>> > wrote:<br>
>> >><br>
>> >> On Tue, Jan 16, 2018 at 12:55 PM, Roman Lebedev <<a href="mailto:lebedev.ri@gmail.com">lebedev.ri@gmail.com</a>><br>
>> >> wrote:<br>
>> >>><br>
>> >>> On Tue, Jan 16, 2018 at 8:35 PM, Nico Weber <<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>><br>
>> >>> wrote:<br>
>> >>> > Wait, wasn't the consensus here to leave the warning out of -Wextra<br>
>> >>> > too?<br>
>> >>> > Looks like r321691 got that wrong?<br>
>> >>> Hm, consensus? Since it was pretty quite clear (?) from the diff and<br>
>> >>> commit<br>
>> >>> message that it was only moved to -Wextra, i guess this was somehow<br>
>> >>> missed by me and all the reviewers...<br>
>> >>><br>
>> >>> On Tue, Jan 16, 2018 at 8:38 PM, Richard Smith via cfe-dev<br>
>> >>> <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>> wrote:<br>
>> >>> > Yes, I'd prefer to take this warning out of -Wextra at least for<br>
>> >>> > Clang<br>
>> >>> > 6.<br>
>> >>> What exactly do you have in mind?<br>
>> >>> Only from release_60 branch, or from trunk+release_60?<br>
>> >><br>
>> >><br>
>> >> trunk+release_60; we do trunk-based development.<br>
>> >><br>
>> >>><br>
>> >>><br>
>> >>> > Hopefully we can find good heuristics to suppress the false<br>
>> >>> > positives<br>
>> >>> > for<br>
>> >>> > Clang 7 and then re-enable it.<br>
>> >>> John McCall did post a short summary of what *seemingly* needs to be<br>
>> >>> done in <a href="https://reviews.llvm.org/D39462#917421" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D39462#917421</a><br>
>> >>> To be honest i'm not quite sure where to start on that.<br>
>> >>><br>
>> >>> Roman.<br>
>> >>><br>
>> >>> > On Tue, Jan 16, 2018 at 12:04 PM, via cfe-dev<br>
>> >>> > <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>><br>
>> >>> > wrote:<br>
>> >>> >><br>
>> >>> >> > -----Original Message-----<br>
>> >>> >> > From: cfe-dev [mailto:<a href="mailto:cfe-dev-bounces@lists.llvm.org">cfe-dev-bounces@lists.<wbr>llvm.org</a>] On Behalf<br>
>> >>> >> > Of<br>
>> >>> >> > Hans<br>
>> >>> >> > Wennborg via cfe-dev<br>
>> >>> >> > Sent: Tuesday, January 16, 2018 8:01 AM<br>
>> >>> >> > To: Roman Lebedev <<a href="mailto:lebedev.ri@gmail.com">lebedev.ri@gmail.com</a>><br>
>> >>> >> > Cc: Marshall Clow <<a href="mailto:mclow.lists@gmail.com">mclow.lists@gmail.com</a>>; Richard Smith<br>
>> >>> >> > <richard-<br>
>> >>> >> > <a href="mailto:llvm@metafoo.co.uk">llvm@metafoo.co.uk</a>>; <a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>; John McCall<br>
>> >>> >> > <<a href="mailto:rjmccall@gmail.com">rjmccall@gmail.com</a>><br>
>> >>> >> > Subject: Re: [cfe-dev] -Wtautological-constant-<wbr>compare issues<br>
>> >>> >> ><br>
>> >>> >> ...<br>
>> >>> >> > >><br>
>> >>> >> > >> Apologies for coming late to the thread.<br>
>> >>> >> > > No problem.<br>
>> >>> >> > ><br>
>> >>> >> > >> What's the status here? Did everything land, and should we<br>
>> >>> >> > >> merge<br>
>> >>> >> > r321691 to 6.0?<br>
>> >>> >> > > It landed before branching. I see r321691 in release_60 branch.<br>
>> >>> >> > > So nothing do be done here.<br>
>> >>> >> ><br>
>> >>> >> > Ah, right. Thanks for checking! Sorry for the noise.<br>
>> >>> >> ><br>
>> >>> >><br>
>> >>> >> Can we consider <a href="https://reviews.llvm.org/D41727" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D41727</a> for inclusion in<br>
>> >>> >> master<br>
>> >>> >> and possibly release_60?<br>
>> >>> >><br>
>> >>> >> -Brian<br>
>> >>> >><br>
>> >>> >> ______________________________<wbr>_________________<br>
>> >>> >> cfe-dev mailing list<br>
>> >>> >> <a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
>> >>> >> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-dev</a><br>
>> >>> ><br>
>> >>> ><br>
>> >><br>
>> >><br>
>> ><br>
><br>
><br>
</div></div></blockquote></div><br></div>