public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH, ARM] Fix broken testcase, vfp-1.c, for Thumb
@ 2011-07-28 15:25 Ian Bolton
  2011-08-04 10:14 ` Richard Earnshaw
  0 siblings, 1 reply; 2+ messages in thread
From: Ian Bolton @ 2011-07-28 15:25 UTC (permalink / raw)
  To: gcc-patches

This patch makes the vfp-1.c testcase work for Thumb.  It became broken when
we
restricted the negative offsets allowed for Thumb to fix up a Spec2K failure
some months back.  (It was previously possible to generate illegal offsets.)

OK for trunk?


Cheers,
Ian


2011-07-28  Ian Bolton  <ian.bolton@arm.com>

testsuite/
	* gcc.target/arm/vfp-1.c: large negative offsets not possible on
Thumb2.



Index: gcc/testsuite/gcc.target/arm/vfp-1.c
===================================================================
--- gcc/testsuite/gcc.target/arm/vfp-1.c        (revision 176838)
+++ gcc/testsuite/gcc.target/arm/vfp-1.c        (working copy)
@@ -127,13 +127,13 @@ void test_convert () {
 
 void test_ldst (float f[], double d[]) {
   /* { dg-final { scan-assembler "flds.+ \\\[r0, #1020\\\]" } } */
-  /* { dg-final { scan-assembler "flds.+ \\\[r0, #-1020\\\]" } } */
+  /* { dg-final { scan-assembler "flds.+ \\\[r\[0-9\], #-1020\\\]" { target
{ arm32 && { ! arm_thumb2_ok } } } } } */
   /* { dg-final { scan-assembler "add.+ r0, #1024" } } */
-  /* { dg-final { scan-assembler "fsts.+ \\\[r0, #0\\\]\n" } } */
+  /* { dg-final { scan-assembler "fsts.+ \\\[r\[0-9\], #0\\\]\n" } } */
   f[256] = f[255] + f[-255];
 
   /* { dg-final { scan-assembler "fldd.+ \\\[r1, #1016\\\]" } } */
-  /* { dg-final { scan-assembler "fldd.+ \\\[r1, #-1016\\\]" } } */
+  /* { dg-final { scan-assembler "fldd.+ \\\[r\[1-9\], #-1016\\\]" { target
{ arm32 && { ! arm_thumb2_ok } } } } } */
   /* { dg-final { scan-assembler "fstd.+ \\\[r1, #256\\\]" } } */
   d[32] = d[127] + d[-127];
 }



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

* Re: [PATCH, ARM] Fix broken testcase, vfp-1.c, for Thumb
  2011-07-28 15:25 [PATCH, ARM] Fix broken testcase, vfp-1.c, for Thumb Ian Bolton
@ 2011-08-04 10:14 ` Richard Earnshaw
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Earnshaw @ 2011-08-04 10:14 UTC (permalink / raw)
  To: Ian Bolton; +Cc: gcc-patches

On 28/07/11 15:50, Ian Bolton wrote:
> This patch makes the vfp-1.c testcase work for Thumb.  It became broken when
> we
> restricted the negative offsets allowed for Thumb to fix up a Spec2K failure
> some months back.  (It was previously possible to generate illegal offsets.)
> 
> OK for trunk?
> 
> 
> Cheers,
> Ian
> 
> 
> 2011-07-28  Ian Bolton  <ian.bolton@arm.com>
> 
> testsuite/
> 	* gcc.target/arm/vfp-1.c: large negative offsets not possible on
> Thumb2.
> 
> 

OK.

R.


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

end of thread, other threads:[~2011-08-04 10:14 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-07-28 15:25 [PATCH, ARM] Fix broken testcase, vfp-1.c, for Thumb Ian Bolton
2011-08-04 10:14 ` Richard Earnshaw

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