[PATCH, GCC/ARM] Define arm_arch_core_flags in a single file

Thomas Preudhomme thomas.preudhomme@foss.arm.com
Fri Dec 9 16:56:00 GMT 2016


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.preudhomme@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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix_arm_arch_core_flags_bloating.patch
Type: text/x-patch
Size: 1774 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20161209/11141daa/attachment.bin>


More information about the Gcc-patches mailing list