public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Patch to fix vector tests on IA64 HP-UX
@ 2007-06-01 20:27 Steve Ellcey
  2007-07-27 20:30 ` Steve Ellcey
  0 siblings, 1 reply; 2+ messages in thread
From: Steve Ellcey @ 2007-06-01 20:27 UTC (permalink / raw)
  To: gcc-patches

A number of vect tests are failing on IA64 HP-UX because we are swapping
vector elements when we shouldn't be.  This patch removes the BIG_ENDIAN
test and does the same thing for both big and little endian platforms.

Tested on IA64 HP-UX and Linux, it caused no regressions on Linux and
on HP-UX it fixes:

	FAIL: gcc.dg/vect/no-tree-dom-vect-bug.c execution test
	FAIL: gcc.dg/vect/pr30784.c execution test
	FAIL: gcc.dg/vect/vect-14.c execution test
	FAIL: gcc.dg/vect/vect-14.c execution test
	FAIL: gcc.dg/vect/vect-iv-10.c execution test
	FAIL: gcc.dg/vect/vect-iv-10.c execution test
	FAIL: gcc.dg/vect/vect-iv-2.c execution test
	FAIL: gcc.dg/vect/vect-iv-2.c execution test
	FAIL: gcc.dg/vect/vect-iv-3.c execution test
	FAIL: gcc.dg/vect/vect-iv-3.c execution test

OK to checkin?  Maybe it is obvious.

Steve Ellcey
sje@cup.hp.com



2007-06-01  Steve Ellcey  <sje@cup.hp.com>

	* config/ia64/vect.md (vec_initv2si): Remove bad BIG_ENDIAN test.

Index: config/ia64/vect.md
===================================================================
--- config/ia64/vect.md	(revision 125253)
+++ config/ia64/vect.md	(working copy)
@@ -802,8 +802,8 @@ (define_expand "vec_initv2si"
   if (GET_CODE (op1) == CONST_INT && GET_CODE (op2) == CONST_INT)
     {
       rtvec v = rtvec_alloc (2);
-      RTVEC_ELT (v, 0) = TARGET_BIG_ENDIAN ? op2 : op1;
-      RTVEC_ELT (v, 1) = TARGET_BIG_ENDIAN ? op1 : op2;;
+      RTVEC_ELT (v, 0) = op1;
+      RTVEC_ELT (v, 1) = op2;
       x = gen_rtx_CONST_VECTOR (V2SImode, v);
       emit_move_insn (operands[0], x);
       DONE;

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Patch to fix vector tests on IA64 HP-UX
  2007-06-01 20:27 Patch to fix vector tests on IA64 HP-UX Steve Ellcey
@ 2007-07-27 20:30 ` Steve Ellcey
  0 siblings, 0 replies; 2+ messages in thread
From: Steve Ellcey @ 2007-07-27 20:30 UTC (permalink / raw)
  To: gcc-patches

| 2007-06-01  Steve Ellcey  <sje@cup.hp.com>
| 
|         * config/ia64/vect.md (vec_initv2si): Remove bad BIG_ENDIAN test.

Since I didn't get any reply to this patch and since it only affects
HP-UX (the only big endian OS on IA64) I am going to check the patch in.
It was tested on HP-UX and Linux IA64.

Steve Ellcey
sje@cup.hp.com

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2007-07-27 20:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-06-01 20:27 Patch to fix vector tests on IA64 HP-UX Steve Ellcey
2007-07-27 20:30 ` Steve Ellcey

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).