[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.

diff --git a/ChangeLog b/ChangeLog
index 50bc0355b7e80fcdbf354a822960d7fdff429433..9d07286d0862d8387dfbbb61b5f1f2fb64205d85 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -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.

diff --git a/sysdeps/powerpc/powerpc32/power4/Makefile b/sysdeps/powerpc/powerpc32/power4/Makefile
index 60aa508ba4c845c3caea6a51116c6e9df250545a..ba06adb5d076bb6147b679c8cae887abee6bbf45 100644 (file)
--- a/sysdeps/powerpc/powerpc32/power4/Makefile
+++ b/sysdeps/powerpc/powerpc32/power4/Makefile
@@ -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

diff --git a/sysdeps/powerpc/powerpc32/power4/fpu/Makefile b/sysdeps/powerpc/powerpc32/power4/fpu/Makefile
index a6fa75ecbc5a5bfdf44d41e1a46c6a22ba389b9d..f487ed60141c4f04dfa93ae825fbae531881b90c 100644 (file)
--- a/sysdeps/powerpc/powerpc32/power4/fpu/Makefile
+++ b/sysdeps/powerpc/powerpc32/power4/fpu/Makefile
@@ -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

diff --git a/sysdeps/powerpc/powerpc64/power4/Makefile b/sysdeps/powerpc/powerpc64/power4/Makefile
index 60aa508ba4c845c3caea6a51116c6e9df250545a..ba06adb5d076bb6147b679c8cae887abee6bbf45 100644 (file)
--- a/sysdeps/powerpc/powerpc64/power4/Makefile
+++ b/sysdeps/powerpc/powerpc64/power4/Makefile
@@ -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

diff --git a/sysdeps/powerpc/powerpc64/power4/fpu/Makefile b/sysdeps/powerpc/powerpc64/power4/fpu/Makefile
index 89dfa5ef35770332766835da0180d723a1cbde37..f8bb3ef04bf3898440f7d6e148cefc44b7ccc850 100644 (file)
--- a/sysdeps/powerpc/powerpc64/power4/fpu/Makefile
+++ b/sysdeps/powerpc/powerpc64/power4/fpu/Makefile
@@ -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