[PATCH (for next stage 1)] Add return type to gimple function dumps

Jeff Law law@redhat.com
Thu Apr 24 21:54:00 GMT 2014


On 03/10/14 13:22, David Malcolm wrote:
> Gimple function dumps contain the types of parameters, but not of the
> return type.
>
> The attached patch fixes this omission; here's an example of the
> before/after diff:
> $ diff -up /tmp/pr23401.c.004t.gimple.old /tmp/pr23401.c.004t.gimple.new
> --- /tmp/pr23401.c.004t.gimple.old      2014-03-10 13:40:08.972063541 -0400
> +++ /tmp/pr23401.c.004t.gimple.new      2014-03-10 13:39:49.346515464 -0400
> @@ -1,3 +1,4 @@
> +int
>   ffff (int i)
>   {
>     int D.1731;
>
>
> Successfully bootstrapped and regrtested on x86_64 Linux (Fedora 20).
>
> A couple of test cases needed tweaking, since they were counting the
> number of occurrences of "int" in the gimple dump, which thus changed
> for functions returning int (like the one above).
>
> OK for next stage 1?
Conceptually OK.  As Richi notes, the work here is in fixing up the 
testsuite.  I didn't see a reply to Richi's question, particularly WRT 
the Fortran testsuite.

jeff



More information about the Gcc-patches mailing list