This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[patch trunk] improving graphite Cloog sorry messages
- From: Basile Starynkevitch <basile at starynkevitch dot net>
- To: gcc-patches at gcc dot gnu dot org
- Date: Wed, 2 Feb 2011 15:41:19 +0100
- Subject: [patch trunk] improving graphite Cloog sorry messages
Hello All,
I find quite confusing the error messages when asking for graphite related
optimization on a GCC where Cloog has been (inadvertently) disabled. I would
like the sorry message to explicitly mention Cloog in that case.
Here is a trivial patch to trunk 169524.
########### gcc/ChangeLog entry
2011-02-02 Basile Starynkevitch <basile@starynkevitch.net>
* toplevel.c (process_options): Mention Cloog in error message
when don't HAVE_cloog.
* graphite.c (graphite_transform_loops): Likewise.
################
Index: gcc/toplev.c
===================================================================
--- gcc/toplev.c (revision 169524)
+++ gcc/toplev.c (working copy)
@@ -1323,7 +1323,7 @@ process_options (void)
sorry ("Graphite loop optimizations cannot be used (-fgraphite, "
"-fgraphite-identity, -floop-block, -floop-flatten, "
"-floop-interchange, -floop-strip-mine, -floop-parallelize-all, "
- "and -ftree-loop-linear)");
+ "and -ftree-loop-linear) without Cloog");
#endif
/* Unrolling all loops implies that standard loop unrolling must also
Index: gcc/graphite.c
===================================================================
--- gcc/graphite.c (revision 169524)
+++ gcc/graphite.c (working copy)
@@ -283,7 +283,7 @@ graphite_transform_loops (void)
void
graphite_transform_loops (void)
{
- sorry ("Graphite loop optimizations cannot be used");
+ sorry ("Graphite loop optimizations cannot be used without Cloog");
}
#endif
################################################################
Comments or Ok are welcome.
Regards.
--
Basile STARYNKEVITCH http://starynkevitch.net/Basile/
email: basile<at>starynkevitch<dot>net mobile: +33 6 8501 2359
8, rue de la Faiencerie, 92340 Bourg La Reine, France
*** opinions {are only mines, sont seulement les miennes} ***