<div dir="ltr">Oh and this patch didn't seems actually correct on Windows because Windows is LLP64 (ICF.cpp expects its value to be 64-bit but this would change it to 32-bit).</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 24, 2015 at 5:17 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">There is some activity in pr24908 and linux 32 bits are probably not<br>
that common (even less with clang), so I would suggest waiting a bit.<br>
<br>
I only did a 32 bit build to try to reproduce an error reported on 32<br>
bit windows :-)<br>
<div class="HOEnZb"><div class="h5"><br>
On 23 September 2015 at 20:15, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
> LGTM as a temporary measure to make it compile with (buggy) clang.<br>
><br>
> On Tue, Sep 22, 2015 at 5:06 AM, Rafael Espíndola<br>
> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>><br>
>> This actually looks like a bug in clang:<br>
>> <a href="https://llvm.org/bugs/show_bug.cgi?id=24908" rel="noreferrer" target="_blank">https://llvm.org/bugs/show_bug.cgi?id=24908</a><br>
>><br>
>><br>
>><br>
>><br>
>> On 22 September 2015 at 07:57, Rafael Espíndola<br>
>> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>> > add llvm-commits.<br>
>> ><br>
>> > On 22 September 2015 at 07:57, Rafael Espíndola<br>
>> > <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>> >> On linux 32 bit the build was failing with<br>
>> >><br>
>> >><br>
>> >> /usr/lib/gcc/x86_64-redhat-linux/4.9.2/../../../../include/c++/4.9.2/bits/atomic_base.h:500:<br>
>> >> error: undefined reference to '__atomic_load_8'<br>
>> >><br>
>> >> /usr/lib/gcc/x86_64-redhat-linux/4.9.2/../../../../include/c++/4.9.2/bits/atomic_base.h:478:<br>
>> >> error: undefined reference to '__atomic_store_8'<br>
>> >><br>
>> >> Cheers,<br>
>> >> Rafael<br>
><br>
><br>
</div></div></blockquote></div><br></div>