<div dir="ltr">Sorry, I reverted commit 220932 by 221331.<div><br></div><div>I have to do this because it is a showstopper failure, and all aarch64 target build are being blocked in the past 4 days.</div><div><br></div><div>Thanks,</div><div>-Jiangning</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2014-11-05 9:15 GMT+08:00 Jiangning Liu <span dir="ltr"><<a href="mailto:liujiangning1@gmail.com" target="_blank">liujiangning1@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Sorry, That was my mistake. It just showed my build can pass after reverting 220932.<br><br>Can this be reverted ASAP? This is actually blocking clang/llvm self-build on aarch64 target. <div><br></div><div>The error message is like,<br><br>[ 20%] Building arm_neon.h...<br>/bin/sh: line 1: 28055 Segmentation fault      ../../../../bin/clang-tblgen -gen-arm-neon -I /work/slave/performance-daily/build/tools/clang/lib/Headers -I /work/slave/performance-daily/build/lib/Target -I /work/slave/performance-daily/build/include /work/slave/performance-daily/build/tools/clang/include/clang/Basic/<a href="http://arm_neon.td" target="_blank">arm_neon.td</a> -o /work/slave/performance-daily/build/build/tools/clang/lib/Headers/arm_neon.h.tmp<br><br>Thanks,<br>-Jiangning<br><div><br></div></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">2014-11-05 9:01 GMT+08:00 Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>----- Original Message -----<br>
> From: "Jiangning Liu" <<a href="mailto:liujiangning1@gmail.com" target="_blank">liujiangning1@gmail.com</a>><br>
> To: "Chris Bieneman" <<a href="mailto:beanz@apple.com" target="_blank">beanz@apple.com</a>><br>
> Cc: "Bill Schmidt" <<a href="mailto:wschmidt@linux.vnet.ibm.com" target="_blank">wschmidt@linux.vnet.ibm.com</a>>, "Samuel F Antao" <<a href="mailto:sfantao@us.ibm.com" target="_blank">sfantao@us.ibm.com</a>>, <a href="mailto:azanella@linux.vnet.ibm.com" target="_blank">azanella@linux.vnet.ibm.com</a>,<br>
> "Hal Finkel" <<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>>, "LLVM Dev" <<a href="mailto:llvmdev@cs.uiuc.edu" target="_blank">llvmdev@cs.uiuc.edu</a>>, "ulrich weigand" <<a href="mailto:ulrich.weigand@de.ibm.com" target="_blank">ulrich.weigand@de.ibm.com</a>><br>
> Sent: Tuesday, November 4, 2014 6:56:07 PM<br>
> Subject: Re: Issue with std::call_once in PPC64 platform<br>
><br>
</span><span>> It seems the crash of llvm/clang build on aarch64 Debian has been<br>
> fixed by r220941.<br>
<br>
</span>That's scary. Is there any actual code-generation change from that commit?<br>
<br>
 -Hal<br>
<div><div><br>
><br>
><br>
> Thanks,<br>
> -Jiangning<br>
><br>
><br>
><br>
><br>
> 2014-11-05 8:45 GMT+08:00 Jiangning Liu < <a href="mailto:liujiangning1@gmail.com" target="_blank">liujiangning1@gmail.com</a> > :<br>
><br>
><br>
><br>
> The versions I'm using right now are<br>
><br>
><br>
> * gcc: (Debian/Linaro 4.9.1-14) 4.9.1<br>
> * libstdc++: libstdc++.so.6.0.20<br>
><br>
><br>
> Thanks,<br>
> -Jiangning<br>
><br>
><br>
><br>
><br>
> 2014-11-05 4:46 GMT+08:00 Chris Bieneman < <a href="mailto:beanz@apple.com" target="_blank">beanz@apple.com</a> > :<br>
><br>
><br>
> Adding Jiangning Liu to the thread.<br>
><br>
> Jiangning reported a similar issue on the llvm-commits list on Debian<br>
> aarch64.<br>
><br>
> In general it sounds like std::call_once may not really be bug free.<br>
><br>
> Jiangning, can you please provide your gcc/libstdc++ version?<br>
><br>
> Thanks,<br>
> -Chris<br>
><br>
><br>
><br>
><br>
> > On Nov 4, 2014, at 9:38 AM, Bill Schmidt <<br>
> > <a href="mailto:wschmidt@linux.vnet.ibm.com" target="_blank">wschmidt@linux.vnet.ibm.com</a> > wrote:<br>
> ><br>
> > On Tue, 2014-11-04 at 12:17 -0500, Samuel F Antao wrote:<br>
> >> Ok, I'll put a patch together to fix this later today. I'll<br>
> >> probably<br>
> >> do what Reid was suggesting and use what is already in there for<br>
> >> Windows.<br>
> ><br>
> > Hm, better hold off on that. Ulrich Weigand has noted that there is<br>
> > invalid code generation occurring around a TLS call. Looks like is<br>
> > is<br>
> > probably a PPC back end problem at the moment.<br>
> ><br>
> > Thanks,<br>
> > Bill<br>
> ><br>
> >><br>
> >> Thanks,<br>
> >> Samuel<br>
> >><br>
> >> Bill Schmidt < <a href="mailto:wschmidt@linux.vnet.ibm.com" target="_blank">wschmidt@linux.vnet.ibm.com</a> > wrote on 11/04/2014<br>
> >> 12:11:08 PM:<br>
> >><br>
> >>> From: Bill Schmidt < <a href="mailto:wschmidt@linux.vnet.ibm.com" target="_blank">wschmidt@linux.vnet.ibm.com</a> ><br>
> >>> To: Samuel F Antao/Watson/IBM@IBMUS<br>
</div></div>> >>> Cc: <a href="mailto:azanella@linux.vnet.ibm.com" target="_blank">azanella@linux.vnet.ibm.com</a> , <a href="mailto:beanz@apple.com" target="_blank">beanz@apple.com</a> , Hal Finkel<br>
<span>> >>> < <a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a> >, LLVM Dev < <a href="mailto:llvmdev@cs.uiuc.edu" target="_blank">llvmdev@cs.uiuc.edu</a> ><br>
> >>> Date: 11/04/2014 12:11 PM<br>
> >>> Subject: Re: Issue with std::call_once in PPC64 platform<br>
> >>><br>
> >>> On Tue, 2014-11-04 at 11:59 -0500, Samuel F Antao wrote:<br>
> >>>> Hi Bill,<br>
> >>>><br>
> >>>> You can find the same issue in the buildbot:<br>
> >>>><br>
> >>>> <a href="http://lab.llvm.org:8011/builders/clang-ppc64-elf-linux2/builds/" target="_blank">http://lab.llvm.org:8011/builders/clang-ppc64-elf-linux2/builds/</a><br>
> >>> 16444/steps/compile.llvm.stage2/logs/stdio<br>
> >>>><br>
> >>>> It is failing for me both in BE (gcc 4.8.2) and LE(4.9.1). I am<br>
> >>>> compiling with clang 3.5, but those are the gcc toolchains I am<br>
> >> using.<br>
> >>>><br>
> >>>> What do you think is the best way to fix this?<br>
> >>><br>
> >>> I don't know what the best fix is, but clearly a fix or revert is<br>
> >> needed<br>
> >>> since the patch is breaking some platforms.<br>
> >>><br>
> >>> Thanks,<br>
> >>> Bill<br>
> >>><br>
> >>>><br>
> >>>> Thanks!<br>
> >>>> Samuel<br>
> >>>><br>
> >>>> Bill Schmidt < <a href="mailto:wschmidt@linux.vnet.ibm.com" target="_blank">wschmidt@linux.vnet.ibm.com</a> > wrote on 11/04/2014<br>
> >>>> 11:45:06 AM:<br>
> >>>><br>
> >>>>> From: Bill Schmidt < <a href="mailto:wschmidt@linux.vnet.ibm.com" target="_blank">wschmidt@linux.vnet.ibm.com</a> ><br>
> >>>>> To: Samuel F Antao/Watson/IBM@IBMUS<br>
</span>> >>>>> Cc: LLVM Dev < <a href="mailto:llvmdev@cs.uiuc.edu" target="_blank">llvmdev@cs.uiuc.edu</a> >, <a href="mailto:beanz@apple.com" target="_blank">beanz@apple.com</a> , Hal<br>
<div><div>> >>>>> Finkel<br>
> >>>>> < <a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a> >, <a href="mailto:azanella@linux.vnet.ibm.com" target="_blank">azanella@linux.vnet.ibm.com</a><br>
> >>>>> Date: 11/04/2014 11:45 AM<br>
> >>>>> Subject: Re: Issue with std::call_once in PPC64 platform<br>
> >>>>><br>
> >>>>> Hi Samuel,<br>
> >>>>><br>
> >>>>> I haven't encountered this problem. What gcc/libstdc++ are you<br>
> >>>> using?<br>
> >>>>> BE or LE?<br>
> >>>>><br>
> >>>>> Copying Adhemerval in case he has heard of any related issues.<br>
> >>>>><br>
> >>>>> Thanks,<br>
> >>>>> Bill<br>
> >>>>><br>
> >>>>> On Mon, 2014-11-03 at 20:19 -0500, Samuel F Antao wrote:<br>
> >>>>>> Hi all,<br>
> >>>>>><br>
> >>>>>> I observe that r220932 (Removing the static initializer in<br>
> >>>>>> ManagedStatic.cpp by using llvm_call_once to initialize the<br>
> >>>>>> ManagedStatic mutex.) is causing tablegen to segfault in PPC<br>
> >>>> platforms<br>
> >>>>>> during static initialization. The crash happens while calling<br>
> >>>>>> std::call_once introduced by this patch in the wrapper used<br>
> >> in<br>
> >>>>>> getManagedStaticMutex.<br>
> >>>>>><br>
> >>>>>> I understand this call is buggy for some platforms (or<br>
> >> probably<br>
> >>>> some<br>
> >>>>>> oldish libstdc++ versions). Is this a known issue? Should we<br>
> >> guard<br>
> >>>>>> LLVM_DEFINE_ONCE_FLAG definition with platform specific macros<br>
> >> to<br>
> >>>>>> avoid the crash, at least for the moment?<br>
> >>>>>><br>
> >>>>>> Thanks!<br>
> >>>>>> Samuel<br>
> >>>>>><br>
> >>>>>><br>
> >>>>><br>
> >>>>><br>
> >>>><br>
> >>>><br>
> >>><br>
> >>><br>
> >><br>
> >><br>
> ><br>
> ><br>
><br>
><br>
><br>
><br>
<br>
</div></div><span><font color="#888888">--<br>
Hal Finkel<br>
Assistant Computational Scientist<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div>