Patchwork [GRAPHITE] Fix the description of graphite_can_represent_loop.

login
register
mail settings
Submitter Vladimir Kargov
Date Aug. 20, 2010, 12:14 a.m.
Message ID <AANLkTikNE-32E87v5gQ098Wab6HeUKTf7UZTVcBfMhm0@mail.gmail.com>
Download mbox | patch
Permalink /patch/62241/
State New
Headers show

Comments

Vladimir Kargov - Aug. 20, 2010, 12:14 a.m.
Hello,

This tiny patch corrects the description of
graphite_can_represent_loop() found in graphite-scop-detection.c.

The description of that function says:
"Return true when it is not possible to represent LOOP in the
polyhedral representation."
However, it clearly does the opposite:

>  if (!graphite_can_represent_expr (scop_entry, loop, niter))
>    return false;
>
>  return true;

-----
2010-08-20  Vladimir Kargov  <kargov@gmail.com>

        * graphite-scop-detection.c (graphite_can_represent_loop): Correct the
        description.
-----

Best regards,
--
Vladimir

Patch

Index: ../gcc/graphite-scop-detection.c
===================================================================
--- ../gcc/graphite-scop-detection.c	(revision 163347)
+++ ../gcc/graphite-scop-detection.c	(working copy)
@@ -384,9 +384,8 @@  harmful_stmt_in_bb (basic_block scop_entry, loop_p
   return NULL;
 }
 
-/* Return true when it is not possible to represent LOOP in the
-   polyhedral representation.  This is evaluated taking SCOP_ENTRY
-   in mind.  */
+/* Return true if LOOP can be represented in the polyhedral representation.
+   This is evaluated taking SCOP_ENTRY in mind.  */
 
 static bool
 graphite_can_represent_loop (basic_block scop_entry, loop_p loop)