<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jan 16, 2018 at 12:55 PM, 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 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 too?<br>
> Looks like r321691 got that wrong?<br>
</span>Hm, consensus? Since it was pretty quite clear (?) from the diff and commit<br>
message that it was only moved to -Wextra, i guess this was somehow<br>
missed by me and all the reviewers...<br>
<span class=""><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 6.<br>
</span>What exactly do you have in mind?<br>
Only from release_60 branch, or from trunk+release_60?<br></blockquote><div><br></div><div>trunk+release_60; we do trunk-based development.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> Hopefully we can find good heuristics to suppress the false positives for<br>
> Clang 7 and then re-enable it.<br>
</span>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>
<span class="HOEnZb"><font color="#888888"><br>
Roman.<br>
</font></span><div class="HOEnZb"><div class="h5"><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 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 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>
</div></div></blockquote></div><br></div></div>