[PowerPC] Remove -ftree-loop-linear from Makefiles

In the past the "-ftree-loop-linear" switch provided a measurable
improvement in performance for certain functions.  At some point it
was assigned as the responsibility of Graphite in GCC.  It has been
found that even with Graphite enabled these flags no longer perform
any appreciable improvement over the baseline.

Graphite now has some open bugs which need to be fixed in order for it
to provide measurable performance improvements but it lacks active
development.  As a result some compiler distributors may disable
Graphite.  If Graphite is disabled then building GLIBC will fail if
the "-ftree-loop-linear" switch is used.

This patch removes the use of "-ftree-loop-linear" as unnecessary.
This commit is contained in:
Tulio Magno Quites Machado Filho 2012-05-18 08:50:33 -05:00 committed by Ryan S. Arnold
parent 610b86225e
commit a88b64b9bb
5 changed files with 18 additions and 6 deletions

View File

@ -1,3 +1,15 @@
2012-05-18 Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com>
* sysdeps/powerpc/powerpc32/power4/Makefile (CFLAGS-wordcopy.c,
CFLAGS-memmove.c): remove -ftree-loop-linear which causes a build
failure if the compiler has Graphite support disabled.
* sysdeps/powerpc/powerpc32/power4/fpu/Makefile (CFLAGS-mpa.c):
Likewise.
* sysdeps/powerpc/powerpc64/power4/Makefile (CFLAGS-wordcopy.c,
CFLAGS-memmove.c): Likewise.
* sysdeps/powerpc/powerpc64/power4/fpu/Makefile (CFLAGS-mpa.c):
Likewise.
2012-05-18 H.J. Lu <hongjiu.lu@intel.com>
* sysdeps/x86_64/x32/_itoa.h: New file.

View File

@ -1,6 +1,6 @@
# Makefile fragment for POWER4/5/5+.
ifeq ($(subdir),string)
CFLAGS-wordcopy.c += --param max-variable-expansions-in-unroller=2 --param max-unroll-times=2 -funroll-loops -fpeel-loops -ftree-loop-linear
CFLAGS-memmove.c += --param max-variable-expansions-in-unroller=2 --param max-unroll-times=2 -funroll-loops -fpeel-loops -ftree-loop-linear
CFLAGS-wordcopy.c += --param max-variable-expansions-in-unroller=2 --param max-unroll-times=2 -funroll-loops -fpeel-loops
CFLAGS-memmove.c += --param max-variable-expansions-in-unroller=2 --param max-unroll-times=2 -funroll-loops -fpeel-loops
endif

View File

@ -1,5 +1,5 @@
# Makefile fragment for POWER4/5/5+ with FPU.
ifeq ($(subdir),math)
CFLAGS-mpa.c += --param max-unroll-times=4 -funroll-loops -fpeel-loops -ftree-loop-linear
CFLAGS-mpa.c += --param max-unroll-times=4 -funroll-loops -fpeel-loops
endif

View File

@ -1,6 +1,6 @@
# Makefile fragment for POWER4/5/5+.
ifeq ($(subdir),string)
CFLAGS-wordcopy.c += --param max-variable-expansions-in-unroller=2 --param max-unroll-times=2 -funroll-loops -fpeel-loops -ftree-loop-linear
CFLAGS-memmove.c += --param max-variable-expansions-in-unroller=2 --param max-unroll-times=2 -funroll-loops -fpeel-loops -ftree-loop-linear
CFLAGS-wordcopy.c += --param max-variable-expansions-in-unroller=2 --param max-unroll-times=2 -funroll-loops -fpeel-loops
CFLAGS-memmove.c += --param max-variable-expansions-in-unroller=2 --param max-unroll-times=2 -funroll-loops -fpeel-loops
endif

View File

@ -1,5 +1,5 @@
# Makefile fragment for POWER4/5/5+ platforms with FPU.
ifeq ($(subdir),math)
CFLAGS-mpa.c += --param max-unroll-times=4 -funroll-loops -fpeel-loops -ftree-loop-linear
CFLAGS-mpa.c += --param max-unroll-times=4 -funroll-loops -fpeel-loops
endif