[PATCH] Kill -fdump-translation-unit

Rainer Orth ro@CeBiTec.Uni-Bielefeld.DE
Thu May 11 17:37:00 GMT 2017


Hi Nathan,

> On 05/10/2017 04:49 AM, Richard Biener wrote:
>> On Tue, May 9, 2017 at 5:41 PM, Nathan Sidwell <nathan@acm.org> wrote:
>>> -fdump-translation-unit is an inscrutably opaque dump.  It turned out that
>>> most of the uses of the tree-dump header file was to indirectly get at
>>> dumpfile.h, and the dump_function entry point it had forwarded to a dumper
>>> in tree-cfg.c.  The gimple dumper would use its node dumper when asked for a
>>> raw dump, but that was about it.
>>>
>>> We have prettier printers now.  This patch nukes the tu dumper.  ok?
>>
>> Ok if nobody objects within 24 hours.
>
> sigh, apparently not.  I committed this to at least clean up the header
> file twistiness.

unfortunately, this part

	* graphite-poly.c: Don't include tree-dump.h.

broke bootstrap with graphite:

/vol/gcc/src/hg/trunk/local/gcc/graphite-poly.c: In function "void new_poly_dr(poly_bb_p, gimple*, poly_dr_type, isl_map*, isl_set*)":
/vol/gcc/src/hg/trunk/local/gcc/graphite-poly.c:108:7: error: "dump_file" was not declared in this scope
   if (dump_file)
       ^

	Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University



More information about the Gcc-patches mailing list