This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [GSoC] type of an isl_ast_expr_id
- From: Tobias Grosser <tobias at grosser dot es>
- To: Roman Gareev <gareevroman at gmail dot com>
- Cc: Mircea Namolaru <mircea dot namolaru at inria dot fr>, gcc-patches at gcc dot gnu dot org
- Date: Wed, 30 Jul 2014 14:40:38 +0200
- Subject: Re: [GSoC] type of an isl_ast_expr_id
- Authentication-results: sourceware.org; auth=none
- References: <CABGF_gf2ggOG8S05nryubkqnsiFysNj6c5-s0gg+iBZv6YGEqA at mail dot gmail dot com> <53D77B7B dot 2090203 at grosser dot es> <CABGF_gdStNVACNpMiOMm1U-HWVv=M2iFE=g4ejF16z99nxO3DA at mail dot gmail dot com> <53D8AE88 dot 1070403 at grosser dot es> <CABGF_gdfn5rMyf=i0atjtQZ0hskTCrRYXxR+4YQqXzQ83ck8Tg at mail dot gmail dot com>
On 30/07/2014 14:32, Roman Gareev wrote:
OK. I proposed a slightly longer description. With the updated description,
you can commit this patch.
I've fixed the fixme.
FIXME: We should replace blind conversation of id's type with
derivation of the optimal type when we get the corresponding isl
support. Blindly converting type sizes may be problematic when we
switch to smaller types.
Would you add anything to it?
Fine with me.
That's what I suggested earlier, no? The only thing I would like you to
check beforehand is if the very same problem still exists with isl trunk.
For this you need to generate an input file for the 'isl_codegen' tool as it
comes with isl that allows to reproduce the bug. The very same input file
can then be submitted to the isl mailing list as a bug report. Could you do
the reporting (and CC me)?
Yes, I wanted to make sure that the code generated by the isl-0.13 is
also not optimal (I sent you the code generated by isl-0.12.2 before).
The report has been sent to the isl mailing list and CC you.
Great. Thanks.
(Sven already replied)
Two comments on your bug report:
1) It would be helpful to also point give the output CLooG generates for
similar input
2) The input is a little involved. If you can provide a minimal test
case, this normally helps in getting bugs fixed faster.
Cheers,
Tobias