<div dir="ltr">Marshall, I'll remove the warning from -Wextra today, I just need to find some time to collect links to all the emails asking for that from various people for the commit message.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 18, 2018 at 9:26 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 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 to<br>
> do it?<br>
</span>To reiterate, i have committed exactly the code that was in the 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>
<div class="HOEnZb"><div class="h5"><br>
> On Tue, Jan 16, 2018 at 12:59 PM, Nico Weber <<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>> 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>> 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 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 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 <<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 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 <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 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>
</div></div></blockquote></div><br></div>