diff mbox

Don't build 32-bit libitm with -march=i486 on x86-64

Message ID 20160419150850.GA17799@intel.com
State New
Headers show

Commit Message

H.J. Lu April 19, 2016, 3:08 p.m. UTC 
Gcc uses the same -march= for both -m32 and -m64 on x86-64 unless
--with-arch-32= is used.  There is no need for -march=i486 to compile
32-bit libitm on x86-64.

Tested on x86-64.  OK for trunk?


H.J.
---
	PR target/70454
	* configure.tgt (XCFLAGS): Don't add -march=i486 to compile
	32-bit target library on x86-64.
---
 libitm/configure.tgt | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)
diff mbox

Patch

diff --git a/libitm/configure.tgt b/libitm/configure.tgt
index e84382f..c925f77 100644
--- a/libitm/configure.tgt
+++ b/libitm/configure.tgt
@@ -100,12 +100,8 @@  case "${target_cpu}" in
 	;;
 
   x86_64)
-	case " ${CC} ${CFLAGS} " in
-	  *" -m32 "*)
-	    XCFLAGS="${XCFLAGS} -march=i486 -mtune=generic"
-	    XCFLAGS="${XCFLAGS} -fomit-frame-pointer"
-	    ;;
-	esac
+	# Since 64-bit arch > i486, we can use the same -march= to build
+	# both 32-bit and 64-bit target libraries.
 	XCFLAGS="${XCFLAGS} -mrtm"
 	ARCH=x86
 	;;