public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Minor typo fixes
@ 2015-08-04  3:53 Jeff Law
  2015-08-05 22:41 ` Abe
  0 siblings, 1 reply; 2+ messages in thread
From: Jeff Law @ 2015-08-04  3:53 UTC (permalink / raw)
  To: gcc-patches; +Cc: Abe Skolnik

[-- Attachment #1: Type: text/plain, Size: 496 bytes --]


I was starting to look at Abe's changes to the gimple if-converter and 
realized a handful of the changes were just fixing comments.  No reason 
those shouldn't go in immediately.

So I pulled them out and applied those changes to the trunk.

Abe -- if you find more of those kind of changes, don't hesitate to 
break them out into their own patch and they can go forward very quickly.

Attached are the fixes that were actually applied.  They were 
bootstrapped for completeness.

Thanks,
Jeff

[-- Attachment #2: patch --]
[-- Type: text/plain, Size: 3134 bytes --]

	* tree-if-conv.c: Fix various typos in comments.
	* tree-vect-stmts.c: Likewise.
 
diff --git a/gcc/tree-if-conv.c b/gcc/tree-if-conv.c
index a284c2a..b5de1b2 100644
--- a/gcc/tree-if-conv.c
+++ b/gcc/tree-if-conv.c
@@ -2283,8 +2283,8 @@ combine_blocks (struct loop *loop, bool any_mask_load_store)
   ifc_bbs = NULL;
 }
 
-/* Version LOOP before if-converting it, the original loop
-   will be then if-converted, the new copy of the loop will not,
+/* Version LOOP before if-converting it; the original loop
+   will be if-converted, the new copy of the loop will not,
    and the LOOP_VECTORIZED internal call will be guarding which
    loop to execute.  The vectorizer pass will fold this
    internal call into either true or false.  */
@@ -2456,7 +2456,7 @@ ifcvt_walk_pattern_tree (tree var, vec<gimple> *defuse_list,
   return;
 }
 
-/* Returns true if STMT can be a root of bool pattern apllied
+/* Returns true if STMT can be a root of bool pattern applied
    by vectorizer.  */
 
 static bool
@@ -2486,7 +2486,7 @@ stmt_is_root_of_bool_pattern (gimple stmt)
   return false;
 }
 
-/*  Traverse all statements in BB which correspondent to loop header to
+/*  Traverse all statements in BB which correspond to loop header to
     find out all statements which can start bool pattern applied by
     vectorizer and convert multiple uses in it to conform pattern
     restrictions.  Such case can occur if the same predicate is used both
@@ -2567,7 +2567,7 @@ ifcvt_local_dce (basic_block bb)
       gimple_set_plf (phi, GF_PLF_2, true);
       worklist.safe_push (phi);
     }
-  /* Consider load/store statemnts, CALL and COND as live.  */
+  /* Consider load/store statements, CALL and COND as live.  */
   for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi); gsi_next (&gsi))
     {
       stmt = gsi_stmt (gsi);
@@ -2655,7 +2655,7 @@ tree_if_conversion (struct loop *loop)
   ifc_bbs = NULL;
   bool any_mask_load_store = false;
 
-  /* Set-up aggressive if-conversion for loops marked with simd pragma.  */
+  /* Set up aggressive if-conversion for loops marked with simd pragma.  */
   aggressive_if_conv = loop->force_vectorize;
   /* Check either outer loop was marked with simd pragma.  */
   if (!aggressive_if_conv)
@@ -2687,7 +2687,7 @@ tree_if_conversion (struct loop *loop)
   combine_blocks (loop, any_mask_load_store);
 
   /* Delete dead predicate computations and repair tree correspondent
-     to bool pattern to delete multiple uses of preidcates.  */
+     to bool pattern to delete multiple uses of predicates.  */
   if (aggressive_if_conv)
     {
       ifcvt_local_dce (loop->header);
diff --git a/gcc/tree-vect-stmts.c b/gcc/tree-vect-stmts.c
index 1256b09..33a7e0c 100644
--- a/gcc/tree-vect-stmts.c
+++ b/gcc/tree-vect-stmts.c
@@ -7125,7 +7125,7 @@ vect_is_simple_cond (tree cond, gimple stmt, loop_vec_info loop_vinfo,
 
    When STMT is vectorized as nested cycle, REDUC_DEF is the vector variable
    to be used at REDUC_INDEX (in then clause if REDUC_INDEX is 1, and in
-   else caluse if it is 2).
+   else clause if it is 2).
 
    Return FALSE if not a vectorizable STMT, TRUE otherwise.  */
 

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

* Re: Minor typo fixes
  2015-08-04  3:53 Minor typo fixes Jeff Law
@ 2015-08-05 22:41 ` Abe
  0 siblings, 0 replies; 2+ messages in thread
From: Abe @ 2015-08-05 22:41 UTC (permalink / raw)
  To: Jeff Law, gcc-patches

Thank you, sir.  :-)

Regards,

Abe




On 8/3/15 10:53 PM, Jeff Law wrote:

> I was starting to look at Abe's changes to the gimple if-converter and realized  a handful
> of the changes were just fixing comments.  No reason those shouldn't go in immediately.
> So I pulled them out and applied those changes to the trunk.

> Abe -- if you find more of those kind of changes, don't hesitate  to break
> them out into their own patch and they can go forward very quickly.

> Attached are the fixes that were actually applied.  They were bootstrapped for completeness.

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

end of thread, other threads:[~2015-08-05 22:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-08-04  3:53 Minor typo fixes Jeff Law
2015-08-05 22:41 ` Abe

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