Re: [Patch][google/main] Fix arm build broken

2014-03-13 Thread 沈涵
Thanks Richard, I'll remove UNSPEC_SIN/COS from my patch. Han On Thu, Mar 13, 2014 at 3:07 AM, Richard Earnshaw wrote: > On 12/03/14 22:35, Hán Shěn (沈涵) wrote: >> ARM build (on chrome) is broken because of duplicate entries in arm.md >> and unspecs.md. Fixed by removing duplication and merge th

Re: [Patch][google/main] Fix arm build broken

2014-03-13 Thread Richard Earnshaw
On 12/03/14 22:35, Hán Shěn (沈涵) wrote: > ARM build (on chrome) is broken because of duplicate entries in arm.md > and unspecs.md. Fixed by removing duplication and merge those in > arm.md into unspecs.md. > > (We had a similar fix for google/gcc-4_8 here - > http://gcc.gnu.org/viewcvs/gcc?view=re

Re: [Patch][google/main] Fix arm build broken

2014-03-12 Thread Dehao Chen
Looks good to me. Dehao On Wed, Mar 12, 2014 at 3:35 PM, Hán Shěn (沈涵) wrote: > ARM build (on chrome) is broken because of duplicate entries in arm.md > and unspecs.md. Fixed by removing duplication and merge those in > arm.md into unspecs.md. > > (We had a similar fix for google/gcc-4_8 here -

[Patch][google/main] Fix arm build broken

2014-03-12 Thread 沈涵
ARM build (on chrome) is broken because of duplicate entries in arm.md and unspecs.md. Fixed by removing duplication and merge those in arm.md into unspecs.md. (We had a similar fix for google/gcc-4_8 here - http://gcc.gnu.org/viewcvs/gcc?view=revision&revision=198650) Tested by building arm cros

Re: [google/main] Fix arm build broken

2014-03-11 Thread Dehao Chen
Looks good to me. Dehao On Tue, Mar 11, 2014 at 3:22 PM, Hán Shěn (沈涵) wrote: > Hi current google/main fails to build for arm because of duplicated > head file entries in gtyp-input.list. > > Fixed by removing duplication in macro tm_file. This only affects arm > platform. Tested by successfully

[google/main] Fix arm build broken

2014-03-11 Thread 沈涵
Hi current google/main fails to build for arm because of duplicated head file entries in gtyp-input.list. Fixed by removing duplication in macro tm_file. This only affects arm platform. Tested by successfully build for arm. Patch below --- config.gcc.orig 2014-03-11 15:10:26.849602409 -0700