diff mbox

[google] Backport r174965 from trunk to google/gcc-4_6 (issue4852046)

Message ID 20110808030018.186602040A@guozhiwei.sha.corp.google.com
State New
Headers show

Commit Message

Carrot Wei Aug. 8, 2011, 3 a.m. UTC
Hi

I want to backport r174965 from trunk to google/gcc-4_6, which fixed vect-72.c
failure in target arm, as described in
http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00927.html

Tested with buildit and regression test on arm qemu.

OK for google/gcc-4_6 ?

thanks
Carrot


2011-08-08  Guozhi Wei  <carrot@google.com>

	Backport r174965 from trunk.

	2011-06-12  Ira Rosen  <ira.rosen@linaro.org>

		* tree-vect-data-refs.c (vect_peeling_hash_get_most_frequent):
		Take number of iterations to peel into account for equally frequent
		misalignment values.




--
This patch is available for review at http://codereview.appspot.com/4852046

Comments

Carrot Wei Aug. 15, 2011, 8:50 a.m. UTC | #1
ping

On Mon, Aug 8, 2011 at 11:00 AM, Guozhi Wei <carrot@google.com> wrote:
>
> Hi
>
> I want to backport r174965 from trunk to google/gcc-4_6, which fixed vect-72.c
> failure in target arm, as described in
> http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00927.html
>
> Tested with buildit and regression test on arm qemu.
>
> OK for google/gcc-4_6 ?
>
> thanks
> Carrot
>
>
> 2011-08-08  Guozhi Wei  <carrot@google.com>
>
>        Backport r174965 from trunk.
>
>        2011-06-12  Ira Rosen  <ira.rosen@linaro.org>
>
>                * tree-vect-data-refs.c (vect_peeling_hash_get_most_frequent):
>                Take number of iterations to peel into account for equally frequent
>                misalignment values.
>
>
> Index: tree-vect-data-refs.c
> ===================================================================
> --- tree-vect-data-refs.c       (revision 177320)
> +++ tree-vect-data-refs.c       (working copy)
> @@ -1250,7 +1250,9 @@ vect_peeling_hash_get_most_frequent (voi
>   vect_peel_info elem = (vect_peel_info) *slot;
>   vect_peel_extended_info max = (vect_peel_extended_info) data;
>
> -  if (elem->count > max->peel_info.count)
> +  if (elem->count > max->peel_info.count
> +      || (elem->count == max->peel_info.count
> +          && max->peel_info.npeel > elem->npeel))
>     {
>       max->peel_info.npeel = elem->npeel;
>       max->peel_info.count = elem->count;
>
>
> --
> This patch is available for review at http://codereview.appspot.com/4852046
Diego Novillo Aug. 15, 2011, 3:27 p.m. UTC | #2
On Sun, Aug 7, 2011 at 23:00, Guozhi Wei <carrot@google.com> wrote:

> 2011-08-08  Guozhi Wei  <carrot@google.com>
>
>        Backport r174965 from trunk.
>
>        2011-06-12  Ira Rosen  <ira.rosen@linaro.org>
>
>                * tree-vect-data-refs.c (vect_peeling_hash_get_most_frequent):
>                Take number of iterations to peel into account for equally frequent
>                misalignment values.

OK.

Diego.
diff mbox

Patch

Index: tree-vect-data-refs.c
===================================================================
--- tree-vect-data-refs.c	(revision 177320)
+++ tree-vect-data-refs.c	(working copy)
@@ -1250,7 +1250,9 @@  vect_peeling_hash_get_most_frequent (voi
   vect_peel_info elem = (vect_peel_info) *slot;
   vect_peel_extended_info max = (vect_peel_extended_info) data;
 
-  if (elem->count > max->peel_info.count)
+  if (elem->count > max->peel_info.count
+      || (elem->count == max->peel_info.count
+          && max->peel_info.npeel > elem->npeel))
     {
       max->peel_info.npeel = elem->npeel;
       max->peel_info.count = elem->count;