[patch] Fix 32 bit build

[Rui Ueyama via llvm-commits]

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).

On Thu, Sep 24, 2015 at 5:17 AM, Rafael Espíndola <
rafael.espindola at gmail.com> wrote:

> There is some activity in pr24908 and linux 32 bits are probably not
> that common (even less with clang), so I would suggest waiting a bit.
>
> I only did a 32 bit build to try to reproduce an error reported on 32
> bit windows :-)
>
> On 23 September 2015 at 20:15, Rui Ueyama <ruiu at google.com> wrote:
> > LGTM as a temporary measure to make it compile with (buggy) clang.
> >
> > On Tue, Sep 22, 2015 at 5:06 AM, Rafael Espíndola
> > <rafael.espindola at gmail.com> wrote:
> >>
> >> This actually looks like a bug in clang:
> >> https://llvm.org/bugs/show_bug.cgi?id=24908
> >>
> >>
> >>
> >>
> >> On 22 September 2015 at 07:57, Rafael Espíndola
> >> <rafael.espindola at gmail.com> wrote:
> >> > add llvm-commits.
> >> >
> >> > On 22 September 2015 at 07:57, Rafael Espíndola
> >> > <rafael.espindola at gmail.com> wrote:
> >> >> On linux 32 bit the build was failing with
> >> >>
> >> >>
> >> >>
> /usr/lib/gcc/x86_64-redhat-linux/4.9.2/../../../../include/c++/4.9.2/bits/atomic_base.h:500:
> >> >> error: undefined reference to '__atomic_load_8'
> >> >>
> >> >>
> /usr/lib/gcc/x86_64-redhat-linux/4.9.2/../../../../include/c++/4.9.2/bits/atomic_base.h:478:
> >> >> error: undefined reference to '__atomic_store_8'
> >> >>
> >> >> Cheers,
> >> >> Rafael
> >
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150924/183f3c0d/attachment.html>