On 09/12/16 16:56, Thomas Preudhomme wrote:
> Hi,
> 
> This patch moves the definition of arm_arch_core_flags along with the
> declaration of its structure element type to
> common/config/arm/arm-common.c so that it is not *defined* in all object
> files including tm.h. Otherwise, GCC gets bloated with as many copy of
> that array as there is file including tm.h and executable as well given
> that crtbegin gets a copy too.
> 
> ChangeLog entry is as follows:
> 
> 
> *** gcc/ChangeLog ***
> 
> 2016-12-09  Thomas Preud'homme  <thomas.preudho...@arm.com>
> 
>         * config/arm/arm-opts.h: Move struct arm_arch_core_flag and
>         arm_arch_core_flags to ...
>         * common/config/arm/arm-common.c: There.
> 
> 
> Testing: successfully built GCC configured with --with-cpu=cortex-a15
> and optional_thumb testcases pass (without any option for -1 and -2,
> with -mcpu=cortex-m3 for -3 to satisfy requirements): build is working
> and feature as well.
> 
> Is this ok for stage3?
> 
> Best regards,
> 
> Thomas

OK.

R.

Reply via email to