Hi Thomas, On 9 December 2016 at 17:56, Thomas Preudhomme <thomas.preudho...@foss.arm.com> 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
I've noticed that this patch (r243541) causes regressions on armeb-none-linux-gnueabihf in arm mode: FAIL:gcc.dg/vect/pr64252.c execution test This is bit surprising given the nature of change... Christophe
