RE: PATCH: PR bootstrap/54209: [4.8 Regression] Failed to build gcc for Android/x86

2012-08-09 Thread Fu, Chao-Ying
> > Where does mips link.h come from? I didn't see it in AOSP > Bionic C library. > > -- > H.J. > It's from development/ndk/platforms/android-9/arch-mips/include/link.h from AOSP checkout. Regards, Chao-ying

RE: PATCH: PR bootstrap/54209: [4.8 Regression] Failed to build gcc for Android/x86

2012-08-09 Thread Fu, Chao-Ying
> Hi, > > Bionic C library doesn't provide link.h. This patch reverts revision > 186788: > > http://gcc.gnu.org/ml/gcc-cvs/2012-04/msg00740.html > > OK to install? > > Thanks. > > H.J. > --- > 2012-08-09 H.J. Lu > > PR bootstrap/54209 > * unwind-dw2-fde-dip.c (USE_PT_GNU_EH_FR

RE: PATCH: PR bootstrap/54209: [4.8 Regression] Failed to build gcc for Android/x86

2012-08-09 Thread Fu, Chao-Ying
> > Hi, > > > > Bionic C library doesn't provide link.h. This patch > reverts revision > > 186788: > > > > http://gcc.gnu.org/ml/gcc-cvs/2012-04/msg00740.html > > > > OK to install? > > > > Thanks. > > > > H.J. > > --- > > 2012-08-09 H.J. Lu > > > > PR bootstrap/54209 > > * unwin

RE: MIPS Android patch

2012-04-23 Thread Fu, Chao-Ying
Richard Sandiford wrote: > > Index: gcc/config/mips/gnu-user.h > > === > > --- gcc/config/mips/gnu-user.h (revision 186580) > > +++ gcc/config/mips/gnu-user.h (working copy) > > @@ -45,8 +45,10 @@ > > /* A standard GNU/Linu

MIPS Android patch

2012-04-19 Thread Fu, Chao-Ying
Hi Maxim, Richard, I built cross-toolchains for 3 different targets as follows. 1. mips-linux-gnu 2. mips-linux-gnu --enable-targets=all 3. mips64-linux-gnu These targets are affected by this MIPS Android patch. Then, I checked the output from "gcc -dumpspecs" before and after applying th

RE: [PATCH, Android] MIPS support

2012-04-17 Thread Fu, Chao-Ying
Maxim Kuvyrkov wrote: > > > >> For now, two MIPS changes in gnu-user.h and > unwind-dw2-fde-dip.c can be posted for comment. > >> (I didn't tested this patch, though.) > > You need to test your patches before posting them for review. > Below are a couple of comments on your current version.

RE: [PATCH] [MIPS] fix mips_prepend insn.

2012-02-03 Thread Fu, Chao-Ying
Liu [pro...@gmail.com] wrote: > OK, I get. > But, sorry, my mips64dspr2 patch has be done... > Should I summit it? I just wonder what version of the MIPS64 DSP/DSPr2 spec that you implemented. Do you have a target CPU that has these MIPS64 DSP/DSPr2 instructions? My concern is that the late

RE: [PATCH] [MIPS] fix mips_prepend insn.

2012-02-02 Thread Fu, Chao-Ying
Richard Sandiford [rdsandif...@googlemail.com] wrote: > This pattern maps directly to __builtin_mips_prepend, which is defined > to take and return an SI type, so :SI is the correct choice here. > > I agree it might be nice to have a function that operates on 64-bit > values for 64-bit targets tho