<p dir="ltr">Hi Tom,</p>
<p dir="ltr">Do you mean the solution I provided at <a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2014-April/071797.html">http://lists.cs.uiuc.edu/pipermail/llvmdev/2014-April/071797.html</a> doesn't work? I will have a look ASAP.</p>

<p dir="ltr">Thanks,<br>
-Jiangning</p>
<div class="gmail_quote">2014-4-10 ÏÂÎç9:27ÓÚ "Tom Stellard" <<a href="mailto:tom@stellard.net">tom@stellard.net</a>>дµÀ£º<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Thu, Apr 10, 2014 at 05:15:50AM +0800, Jiangning Liu wrote:<br>
> Hi Tom,<br>
><br>
> I'm extremely appreciative of your help on back porting those patches!<br>
><br>
> > > 198937 Make sure -use-init-array has intended effect on all AArch64 ELF<br>
> > > targets, not just linux.<br>
> ><br>
> > I wasn't able to merge this commit, because it broke ABI compatibility of<br>
> > the LLVM library shared object.  This is because it removes the function<br>
> > llvm::AArch64LinuxTargetObjectFileInitialize ( llvm::MCContext&,<br>
> >                                                llvm:TargetMachine const&);<br>
> ><br>
> > Does the clang commit mentioned above require this LLVM change?  I<br>
> > wasn't sure, so I didn't merge it.  If you can come up with a version of<br>
> > this patch that doesn't break the ABI, then I can merge it.  Let me know<br>
> > what you want to do.<br>
> ><br>
> Yes, this patch is required by the clang commit. The patch is to change the<br>
> name only, so can you simply keep two methods before and after the<br>
> replacement and make them to have the same content, I think it should work<br>
> and will not break ABI.<br>
><br>
<br>
I'm having trouble porting the clang commit r198940.  It causes two<br>
tests to fail:<br>
<br>
Clang :: Driver/bindings.c<br>
Clang :: Driver/unknown-gcc-arch.c<br>
<br>
Would you be able to prepare a working patch that I can apply to the<br>
stable branch?<br>
<br>
Thanks,<br>
Tom<br>
</blockquote></div>