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.