about summary refs log tree commit diff
diff options
context:
space:
mode:
authorTulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com>2012-05-18 08:50:33 -0500
committerRyan S. Arnold <rsa@linux.vnet.ibm.com>2012-05-18 10:15:30 -0500
commita88b64b9bbf8b2a315206e211250c5529534a477 (patch)
treefa8db8f25494143a6a4396026e5243a514cd50a0
parent610b86225e8d8ebd1400b4c083a04b624b4417b1 (diff)
downloadglibc-a88b64b9bbf8b2a315206e211250c5529534a477.tar.gz
glibc-a88b64b9bbf8b2a315206e211250c5529534a477.tar.xz
glibc-a88b64b9bbf8b2a315206e211250c5529534a477.zip
[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.
-rw-r--r--ChangeLog12
-rw-r--r--sysdeps/powerpc/powerpc32/power4/Makefile4
-rw-r--r--sysdeps/powerpc/powerpc32/power4/fpu/Makefile2
-rw-r--r--sysdeps/powerpc/powerpc64/power4/Makefile4
-rw-r--r--sysdeps/powerpc/powerpc64/power4/fpu/Makefile2
5 files changed, 18 insertions, 6 deletions
diff --git a/ChangeLog b/ChangeLog
index 50bc0355b7..9d07286d08 100644
--- 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 60aa508ba4..ba06adb5d0 100644
--- 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 a6fa75ecbc..f487ed6014 100644
--- 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 60aa508ba4..ba06adb5d0 100644
--- 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 89dfa5ef35..f8bb3ef04b 100644
--- 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