[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