[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