This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [GSoC] the separate option for all dimensions
- From: Roman Gareev <gareevroman at gmail dot com>
- To: Tobias Grosser <tobias at grosser dot es>
- Cc: Mircea Namolaru <mircea dot namolaru at inria dot fr>, gcc-patches at gcc dot gnu dot org
- Date: Wed, 6 Aug 2014 21:21:28 +0600
- Subject: Re: [GSoC] the separate option for all dimensions
- Authentication-results: sourceware.org; auth=none
- References: <CABGF_gcUVb5z7GRf2Vg0yKojcGbX_U0TJbVsjrtVOEGRa_EU3A at mail dot gmail dot com> <53E07796 dot 1070301 at grosser dot es>
I've tested the modified version of Graphite using the gcc test suite
and haven't found out new failed tests.
However, pr35356-2.c is still not suitable for testing. The ISL AST
generated from its source code doesn't contain MIN or MAX.
if (k <= -1) {
for (int c1 = 0; c1 < n; c1 += 1)
S_7(c1);
} else if (k >= n) {
for (int c1 = 0; c1 < n; c1 += 1)
S_7(c1);
} else {
for (int c1 = 0; c1 < k; c1 += 1)
S_7(c1);
S_6(k);
for (int c1 = k + 1; c1 < n; c1 += 1)
S_7(c1);
}
Should we make pr35356-2.c to be similar to isl-codegen-loop-dumping.c
by replacing âMIN_EXPR\[^\\n\\r]*;" and "MAX_EXPR\[^\\n\\r]*;" with
the regexp, which contains the the above-mentioned isl ast?
--
Cheers, Roman Gareev.