diff mbox

[testsuite] Refine scanning string in pr65447.c to support small address offset target

Message ID 000001d09ce8$135270a0$39f751e0$@arm.com
State New
Headers show

Commit Message

Bin Cheng June 2, 2015, 3:56 a.m. UTC 
Hi,
On some arm processors, the offset supported in addressing modes is very
small.  As a result, the dozens of address induction variables will be
grouped into several groups, rather than only one as on armv7/8.  This patch
refines scanning string to avoid test failure on such processors.

It's an obvious change, and test acts as expected.  So is it OK?  I will
commit it in next 24 hours if there is no objection.

Thanks,
bin

2015-06-02  Bin Cheng  <bin.cheng@arm.com>

	PR tree-optimization/65447
	* gcc.dg/tree-ssa/pr65447.c: Increase searching number.

Comments

Bernhard Reutner-Fischer June 2, 2015, 3:40 p.m. UTC | #1 
On June 2, 2015 5:56:13 AM GMT+02:00, Bin Cheng <bin.cheng@arm.com> wrote:
>Hi,
>On some arm processors, the offset supported in addressing modes is
>very
>small.  As a result, the dozens of address induction variables will be
>grouped into several groups, rather than only one as on armv7/8.  This
>patch
>refines scanning string to avoid test failure on such processors.
>
>It's an obvious change, and test acts as expected.  So is it OK?  I
>will
>commit it in next 24 hours if there is no objection.
>
>Thanks,
>bin
>
>2015-06-02  Bin Cheng  <bin.cheng@arm.com>
>
>	PR tree-optimization/65447
>	* gcc.dg/tree-ssa/pr65447.c: Increase searching number.

There should be no cleanup-tree-dump left on trunk.
Please refresh your patch before pushing when somebody OKs it.

Thanks,
Bin.Cheng June 3, 2015, 6:07 a.m. UTC | #2 
On Tue, Jun 2, 2015 at 11:40 PM, Bernhard Reutner-Fischer
<rep.dot.nop@gmail.com> wrote:
> On June 2, 2015 5:56:13 AM GMT+02:00, Bin Cheng <bin.cheng@arm.com> wrote:
>>Hi,
>>On some arm processors, the offset supported in addressing modes is
>>very
>>small.  As a result, the dozens of address induction variables will be
>>grouped into several groups, rather than only one as on armv7/8.  This
>>patch
>>refines scanning string to avoid test failure on such processors.
>>
>>It's an obvious change, and test acts as expected.  So is it OK?  I
>>will
>>commit it in next 24 hours if there is no objection.
>>
>>Thanks,
>>bin
>>
>>2015-06-02  Bin Cheng  <bin.cheng@arm.com>
>>
>>       PR tree-optimization/65447
>>       * gcc.dg/tree-ssa/pr65447.c: Increase searching number.
>
> There should be no cleanup-tree-dump left on trunk.
> Please refresh your patch before pushing when somebody OKs it.
Thanks for reminding.  I am aware of that.  Patch committed as revision 224055.

Thanks,
bin
>
> Thanks,
>
>
>
>
diff mbox

Patch

Index: gcc/testsuite/gcc.dg/tree-ssa/pr65447.c
===================================================================
--- gcc/testsuite/gcc.dg/tree-ssa/pr65447.c	(revision 223794)
+++ gcc/testsuite/gcc.dg/tree-ssa/pr65447.c	(working copy)
@@ -50,5 +50,5 @@  void foo (double *p)
 }
 
 /* We should groups address type IV uses.  */
-/* { dg-final { scan-tree-dump-not "\\nuse 2\\n" "ivopts" } }  */
+/* { dg-final { scan-tree-dump-not "\\nuse 4\\n" "ivopts" } }  */
 /* { dg-final { cleanup-tree-dump "ivopts" } }  */