On 08/17/2016 02:23 AM, Richard Biener wrote:
On Tue, Aug 16, 2016 at 6:06 PM, Jeff Law <l...@redhat.com> wrote:
On 08/16/2016 08:01 AM, Tamar Christina wrote:
Hi All,
This patch turns off the usage of section anchors for
declarations that do not fit in a single anchor range.
A large enough object will use the full anchor range
and also force the use of another anchor pointer.
By not using an anchor for large objects more globals
can share the same anchor.
The patch has been benchmarked using Spec2000 and
the impact on performance is negligible, however some files
using large arrays showed a appreciable reduction in amount of
instructions in the assembly file.
Regression tests were run on aarch64-none-elf and no regressions.
Ok for trunk?
2016-08-16 Tamar Christina <tamar.christ...@arm.com>
Ramana Radhakrishnan <ramana.radhakrish...@arm.com>
* gcc/varasm.c
(default_use_anchors_for_symbol_p): Reject too large decls.
Do you have to worry about DECL_SIZE being non-constant here or are those
filtered out earlier?
Globals can't have variable size.
Duh. You're obviously correct.
Patch is OK for the trunk.
jeff
