[libc++] Patch for AIX port

Xing Xue xingxue at ca.ibm.com
Thu Aug 15 11:18:11 PDT 2013


We may use libc++ on other platforms in addition to AIX.  So, instead of
having one triple for each platform that IBM C++ compiler supports, IBM
specific stuff is grouped under triple ibm.  Notice "linux" is under -ibm-.
Changing -ibm- to -aix- does not make sense.

+    *-ibm-*)
+        if [ $hostOS == "linux" ]
+        then
+           rm -f libc++.so
+           ln -s libc++.so.1.0 libc++.so
+        else #AIX
+           rm -f libc++.a
+           ar r libc++.a shr.o
+        fi
+        ;;

Thanks,
Xing


On Thu, Aug 15, 2013 at 10:51:08AM -0400, Howard Hinnant wrote:
> On Aug 15, 2013, at 7:20 AM, Joerg Sonnenberger <joerg at britannica.bec.de>
wrote:
>
> > On Tue, Aug 13, 2013 at 08:59:13PM -0400, Howard Hinnant wrote:
> >> diff -ruN libcxx.orig/lib/buildit libcxx/lib/buildit
> >> --- libcxx.orig/lib/buildit		 2013-08-13 15:51:00.510247093
-0400
> >> +++ libcxx/lib/buildit		 2013-08-12 17:45:00.180244845 -0400
> >> @@ -136,13 +153,23 @@
> >>         ;;
> >>     *-*-mingw*)
> >>         ;;
> >> +    *-ibm-*)
> >> +        if [ $hostOS == "linux" ]
> >> +        then
> >> +           rm -f libc++.so
> >> +           ln -s libc++.so.1.0 libc++.so
> >> +        else #AIX
> >> +           rm -f libc++.a
> >> +           ar r libc++.a shr.o
> >> +        fi
> >> +        ;;
> >>     *)
> >>         rm -f libc++.so
> >> -        ln -s libc++.so.1 libc++.so
> >> +        ln -s libc++.so.1.0 libc++.so
> >>         ;;
> >> esac
> >
> > Why is this matching the vendor at all?
>
> I can speak for -apple-:
>
> Because that's the way I need it built and it isn't what others need.

The Apple branches all match the OS part of the triple, not the vendor?
What I mean is: shouldn't this be *-*-aix or whatever.

Joerg
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130815/3c42b466/attachment.html>


More information about the cfe-commits mailing list