public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* VEC re-write [patch 12/25]
@ 2012-11-15 21:55 Diego Novillo
  2012-11-16 23:20 ` Ian Lance Taylor
  0 siblings, 1 reply; 3+ messages in thread
From: Diego Novillo @ 2012-11-15 21:55 UTC (permalink / raw)
  To: bernds, gcc-patches

2012-11-15  Diego Novillo  <dnovillo@google.com>

	Adjust for new vec API (http://gcc.gnu.org/wiki/cxx-conversion/cxx-vec)

	* config/bfin/bfin.c: Use new vec API in vec.h.

diff --git a/gcc/config/bfin/bfin.c b/gcc/config/bfin/bfin.c
index 2c01cf7..f2d8473 100644
--- a/gcc/config/bfin/bfin.c
+++ b/gcc/config/bfin/bfin.c
@@ -3480,8 +3480,8 @@ hwloop_optimize (hwloop_info loop)
       insn = BB_END (loop->incoming_src);
       /* If we have to insert the LSETUP before a jump, count that jump in the
 	 length.  */
-      if (VEC_length (edge, loop->incoming) > 1
-	  || !(VEC_last (edge, loop->incoming)->flags & EDGE_FALLTHRU))
+      if (vec_safe_length (loop->incoming) > 1
+	  || !(loop->incoming->last ()->flags & EDGE_FALLTHRU))
 	{
 	  gcc_assert (JUMP_P (insn));
 	  insn = PREV_INSN (insn);
@@ -3749,8 +3749,8 @@ hwloop_optimize (hwloop_info loop)
   if (loop->incoming_src)
     {
       rtx prev = BB_END (loop->incoming_src);
-      if (VEC_length (edge, loop->incoming) > 1
-	  || !(VEC_last (edge, loop->incoming)->flags & EDGE_FALLTHRU))
+      if (vec_safe_length (loop->incoming) > 1
+	  || !(loop->incoming->last ()->flags & EDGE_FALLTHRU))
 	{
 	  gcc_assert (JUMP_P (prev));
 	  prev = PREV_INSN (prev);

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

* Re: VEC re-write [patch 12/25]
  2012-11-15 21:55 VEC re-write [patch 12/25] Diego Novillo
@ 2012-11-16 23:20 ` Ian Lance Taylor
  2012-11-16 23:28   ` Jeff Law
  0 siblings, 1 reply; 3+ messages in thread
From: Ian Lance Taylor @ 2012-11-16 23:20 UTC (permalink / raw)
  To: Diego Novillo; +Cc: bernds, gcc-patches

On Thu, Nov 15, 2012 at 1:53 PM, Diego Novillo <dnovillo@google.com> wrote:
> 2012-11-15  Diego Novillo  <dnovillo@google.com>
>
>         Adjust for new vec API (http://gcc.gnu.org/wiki/cxx-conversion/cxx-vec)
>
>         * config/bfin/bfin.c: Use new vec API in vec.h.

I think that changes like this count as obvious and do not require
separate approval if the rest of the patches are approved.

Ian

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

* Re: VEC re-write [patch 12/25]
  2012-11-16 23:20 ` Ian Lance Taylor
@ 2012-11-16 23:28   ` Jeff Law
  0 siblings, 0 replies; 3+ messages in thread
From: Jeff Law @ 2012-11-16 23:28 UTC (permalink / raw)
  To: Ian Lance Taylor; +Cc: Diego Novillo, bernds, gcc-patches

On 11/16/2012 04:19 PM, Ian Lance Taylor wrote:
> On Thu, Nov 15, 2012 at 1:53 PM, Diego Novillo <dnovillo@google.com> wrote:
>> 2012-11-15  Diego Novillo  <dnovillo@google.com>
>>
>>          Adjust for new vec API (http://gcc.gnu.org/wiki/cxx-conversion/cxx-vec)
>>
>>          * config/bfin/bfin.c: Use new vec API in vec.h.
>
> I think that changes like this count as obvious and do not require
> separate approval if the rest of the patches are approved.
Agreed.
jeff

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

end of thread, other threads:[~2012-11-16 23:28 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-11-15 21:55 VEC re-write [patch 12/25] Diego Novillo
2012-11-16 23:20 ` Ian Lance Taylor
2012-11-16 23:28   ` Jeff Law

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