This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [GSoC] Addition of ISL AST generation to Graphite
- From: Sebastian Pop <sebpop at gmail dot com>
- To: Tobias Grosser <tobias at grosser dot es>
- Cc: Roman Gareev <gareevroman at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Mircea Namolaru <mircea dot namolaru at inria dot fr>
- Date: Mon, 23 Jun 2014 02:29:04 -0700
- Subject: Re: [GSoC] Addition of ISL AST generation to Graphite
- Authentication-results: sourceware.org; auth=none
- References: <CABGF_gcisNL2R2vVd8A1KaG1gcgOUok0EekK=8PTtkxg_VHU5w at mail dot gmail dot com> <53A1E2B9 dot 3090205 at grosser dot es>
Please add a FIXME note in graphite_regenerate_ast_isl saying that
this is not yet a full implementation of the code generator with ISL
ASTs.
It would be useful to make the current graphite_regenerate_ast_isl
working by calling graphite_regenerate_ast_cloog and adding the fixme
note above saying that we rely on the cloog code generator until we
implement the ISL AST parsing.
There is also a minor code style issue in:
isl_set * context_isl = isl_set_params (isl_set_copy (scop->context));
please remove the space after *:
isl_set *context_isl = isl_set_params (isl_set_copy (scop->context));
Otherwise the two patches look good to me.
Thanks,
Sebastian
On Wed, Jun 18, 2014 at 12:04 PM, Tobias Grosser <tobias@grosser.es> wrote:
> On 18/06/2014 21:00, Roman Gareev wrote:
>>
>> These patches add ISL AST generation to graphite, which can be chosen
>> by the fgraphite-code-generator=[isl|cloog] switch. The first patch
>> makes initial renaming of gloog and gloog_error to
>> graphite_regenerate_ast_cloog and graphite_regenerate_error,
>> respectively. The second one adds new files with generation of ISL
>> AST, new switch, new testcase that checks that the dump is generated.
>>
>> Is it fine for trunk?
>
>
> I went over this from the graphite side and it looks fine. However,
> as I did not commit for a while to gcc, it would be great if someone else
> could have a look.
>
> Cheers,
> Tobias
>