From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 14051 invoked by alias); 7 Apr 2011 19:42:28 -0000 Received: (qmail 14043 invoked by uid 22791); 7 Apr 2011 19:42:28 -0000 X-SWARE-Spam-Status: No, hits=-1.6 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE X-Spam-Check-By: sourceware.org Received: from smtp-154-thursday.nerim.net (HELO maiev.nerim.net) (194.79.134.154) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 07 Apr 2011 19:42:23 +0000 Received: from hector.lesours (ours.starynkevitch.net [213.41.244.95]) by maiev.nerim.net (Postfix) with ESMTPS id 3EB7C2E008 for ; Thu, 7 Apr 2011 21:42:22 +0200 (CEST) Received: from glinka.lesours ([192.168.0.1]) by hector.lesours with smtp (Exim 4.74) (envelope-from ) id 1Q7v5o-0002ZL-8r; Thu, 07 Apr 2011 21:42:12 +0200 Date: Thu, 07 Apr 2011 19:42:00 -0000 From: Basile Starynkevitch To: gcc-patches@gcc.gnu.org Subject: Patch: add {tree,gimple}-pretty-print.h & realmpfr.h to PLUGIN_HEADERS Message-Id: <20110407214118.2b121fe9.basile@starynkevitch.net> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2011-04/txt/msg00574.txt.bz2 Hello All, The following tiny patch add some files to PLUGIN_HEADERS. Since they are missing in 4.6, I had to copy them in the MELT plugin tar ball release candidate 0. ##### tiny patch to trunk 172124 Index: gcc/Makefile.in =================================================================== --- gcc/Makefile.in (revision 172124) +++ gcc/Makefile.in (working copy) @@ -4534,6 +4534,7 @@ $(EXCEPT_H) tree-ssa-sccvn.h real.h output.h $(IPA_UTILS_H) \ $(C_PRAGMA_H) $(CPPLIB_H) $(FUNCTION_H) \ cppdefault.h flags.h $(MD5_H) params.def params.h prefix.h tree-inline.h \ + gimple-pretty-print.h tree-pretty-print.h realmpfr.h \ $(IPA_PROP_H) $(RTL_H) $(TM_P_H) $(CFGLOOP_H) $(EMIT_RTL_H) version.h # generate the 'build fragment' b-header-vars ###### gcc/ChangeLog entry 2011-04-07 Basile Starynkevitch * Makefile.in (PLUGIN_HEADERS): Add gimple-pretty-print.h tree-pretty-print.h & realmpfr.h. ###### Some more explanations about why I feel these files are badly needed for plugins (not only for MELT as a plugin, but for other plugins). Pretty printing routines are very useful to help understand, debug, and dump any additional passes, so plugin providing passes working on Gimple or Tree need the gimple-pretty-print.h & tree-pretty-print.h, since to call dumping or debug routines like debug_c_tree or print_declaration or dump_gimple_stmt you need to include these files. Please consider that people coding plugins know Gimple & Tree much less than GCC top level experts, and they are much more likely to need debug or dumping routines! The realmpfr.h file is needed to operate on tree or gimple containing real constants. For instance, a plugin to find all occurrences (in Gimple) of 3.14159 need it, or also a plugin which constant-fold the calls to which are not already constant folded in GCC, or a plugin which constant-fold calls to some other (but less standard) numerical library. Ok for trunk? Cheers. PS. If I am lucky enough to have this patch accepted for trunk, I might even later consider proposing to backport it to gcc 4.6.1. -- Basile STARYNKEVITCH http://starynkevitch.net/Basile/ email: basilestarynkevitchnet mobile: +33 6 8501 2359 8, rue de la Faiencerie, 92340 Bourg La Reine, France *** opinions {are only mine, sont seulement les miennes} ***