<div dir="ltr">I'd probably test to see if putting __attribute__((used)) on them is enough to suppress the warning. Otherwise probably move the class outside the function so it's no longer local.</div><br><div class="gmail_quote"><div dir="ltr">On Sat, Jun 10, 2017 at 4:33 PM Francis Ricci <<a href="mailto:francisjricci@gmail.com">francisjricci@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Looping in David on this (regards warnings introduced by our reverse iterator test)<br><br><div class="gmail_quote"><div dir="ltr">On Sat, Jun 10, 2017, 7:21 PM Davide Italiano <<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">FWIW, a not-so-recent Fedora (1.5 years ago) says:<br>
<br>
[davide@cupiditate ~]$ rpm -q libstdc++<br>
libstdc++-6.3.1-1.fc24.x86_64<br>
libstdc++-6.3.1-1.fc24.i686<br>
<br>
>From what I understand we still support GCC 4.8 but maybe with GCC 7<br>
out we might consider bumping our minimum supported version?<br>
<br>
--<br>
Davide<br>
<br>
On Sat, Jun 10, 2017 at 4:13 PM, Davide Italiano <<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>> wrote:<br>
> Ugh, sorry, spoke too soon.<br>
> Apparently some relatively ancient version of libstdc++ needs these.<br>
><br>
> <a href="http://lab.llvm.org:8011/builders/clang-x86_64-linux-selfhost-modules-2/builds/8386/steps/compile/logs/stdio" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/clang-x86_64-linux-selfhost-modules-2/builds/8386/steps/compile/logs/stdio</a><br>
><br>
> --<br>
> Davide<br>
><br>
> On Sat, Jun 10, 2017 at 4:05 PM, Francis Ricci <<a href="mailto:francisjricci@gmail.com" target="_blank">francisjricci@gmail.com</a>> wrote:<br>
>> Thanks Davide.<br>
>><br>
>> On Sat, Jun 10, 2017, 7:02 PM Davide Italiano <<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>> wrote:<br>
>>><br>
>>> On Sat, Jun 10, 2017 at 3:54 PM, Galina Kistanova via llvm-commits<br>
>>> <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
>>> > Hello Francis,<br>
>>> ><br>
>>> > Could you fix the warnings added by this commit, please?<br>
>>> ><br>
>>><br>
>>> r305157. FWIW, it's generally easier to just fix when the time needed<br>
>>> to write the mail is equivalent to the time of writing the patch, Galina<br>
>>> :)<br>
>>><br>
>>> --<br>
>>> Davide<br>
><br>
><br>
><br>
> --<br>
> Davide<br>
><br>
> "There are no solved problems; there are only problems that are more<br>
> or less solved" -- Henri Poincare<br>
<br>
<br>
<br>
--<br>
Davide<br>
<br>
"There are no solved problems; there are only problems that are more<br>
or less solved" -- Henri Poincare<br>
</blockquote></div>
</blockquote></div>