<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Wed, Jul 6, 2016 at 4:09 PM Davide Italiano via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Jul 6, 2016 at 11:44 AM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>> wrote:<br>
> On Tue, Jul 5, 2016 at 5:19 PM Davide Italiano via llvm-commits<br>
> <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
>><br>
>> On Mon, Jun 27, 2016 at 4:26 PM, Chandler Carruth via llvm-commits<br>
>> <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
>> > Author: chandlerc<br>
>> > Date: Mon Jun 27 18:26:08 2016<br>
>> > New Revision: 273961<br>
>> ><br>
>> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=273961&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=273961&view=rev</a><br>
>> > Log:<br>
>> > [PM] Improve the debugging and logging facilities of the CGSCC bits of<br>
>> > the new pass manager.<br>
>> ><br>
>> > This adds operator<< overloads for the various bits of the<br>
>> > LazyCallGraph, dump methods for use from the debugger, and debug logging<br>
>> > using them to the CGSCC pass manager.<br>
>> ><br>
>> > Having this was essential for debugging the call graph update patch, and<br>
>> > I've extracted what I could from that patch here to minimize the delta.<br>
>> ><br>
>> > Modified:<br>
>> >     llvm/trunk/include/llvm/Analysis/CGSCCPassManager.h<br>
>> >     llvm/trunk/include/llvm/Analysis/LazyCallGraph.h<br>
>> >     llvm/trunk/lib/Analysis/LazyCallGraph.cpp<br>
>> >     llvm/trunk/lib/Passes/PassBuilder.cppHi Hi<br>
>> >     llvm/trunk/test/Other/new-pass-manager.ll<br>
>> ><br>
>><br>
>> Hi Chandler, this breaks the build with GCC6. Can you please take a look?<br>
><br>
><br>
> Yikes, just back from the holiday weekend and looking at this. Is there a<br>
> build bot with this configuration? I'm worried I may not get it on my first<br>
> shot...<br>
><br>
<br>
After a discussion with Chandler on IRC, we agreed this is a bug in<br>
GCC. No bot uses GCC6 yet but we might want to consider disabling the<br>
warning to avoid false positives like this one.<br></blockquote><div><br></div><div>Well, I'd really like it to be warning clean so I've gone ahead and applied the same change Sean did for another case to these friends. Let me know if this helps.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks,<br>
<br>
--<br>
Davide<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div>