Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-20 Thread Richard Biener
On Tue, Nov 19, 2013 at 10:57 PM, Basile Starynkevitch wrote: > On Tue, 2013-11-19 at 20:33 +0100, Dominique Dhumieres wrote: >> > Thanks: Committed revision 205038. >> >> This seems to break several g++ tests: see >> http://gcc.gnu.org/ml/gcc-testresults/2013-11/msg01482.html >> On x86_64-apple-

Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-19 Thread Dominique Dhumieres
Basile, The patch fixes most of the plugin failures except: FAIL: g++.dg/plugin/selfassign.c compilation The error is /opt/gcc/work/gcc/testsuite/g++.dg/plugin/selfassign.c: In function 'void warn_self_assign(gimple)': /opt/gcc/work/gcc/testsuite/g++.dg/plugin/selfassign.c:227:75: error: 'may

Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-19 Thread Basile Starynkevitch
On Tue, 2013-11-19 at 20:33 +0100, Dominique Dhumieres wrote: > > Thanks: Committed revision 205038. > > This seems to break several g++ tests: see > http://gcc.gnu.org/ml/gcc-testresults/2013-11/msg01482.html > On x86_64-apple-darwin13, the ICE is > > cc1: error: cannot load plugin ./one_time_p

Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-19 Thread Dominique Dhumieres
Basile, http://gcc.gnu.org/ml/gcc-testresults/2013-11/msg01482.html is for x86_64-unknown-linux-gnu. However I cannot infer from the pointer what is the ICE. Nevertheless I'll try your patch. Thanks for the answer. Dominique

Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-19 Thread Basile Starynkevitch
On Tue, 2013-11-19 at 20:33 +0100, Dominique Dhumieres wrote: > > Thanks: Committed revision 205038. > > This seems to break several g++ tests: see > http://gcc.gnu.org/ml/gcc-testresults/2013-11/msg01482.html > On x86_64-apple-darwin13, the ICE is I would guess that the following trivial patch

Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-19 Thread Basile Starynkevitch
On Tue, 2013-11-19 at 20:33 +0100, Dominique Dhumieres wrote: > > Thanks: Committed revision 205038. > > This seems to break several g++ tests: see > http://gcc.gnu.org/ml/gcc-testresults/2013-11/msg01482.html > On x86_64-apple-darwin13, the ICE is > > cc1: error: cannot load plugin ./one_time_p

Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-19 Thread Dominique Dhumieres
> Thanks: Committed revision 205038. This seems to break several g++ tests: see http://gcc.gnu.org/ml/gcc-testresults/2013-11/msg01482.html On x86_64-apple-darwin13, the ICE is cc1: error: cannot load plugin ./one_time_plugin.so dlopen(./one_time_plugin.so, 10): image not found /opt/gcc/work/gcc

Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-19 Thread Basile Starynkevitch
On Tue, 2013-11-19 at 16:16 +, Joseph S. Myers wrote: > On Tue, 19 Nov 2013, Basile Starynkevitch wrote: > > > Thanks for your attention. I am attaching a slightly improved patch > > against trunk svn rev. 305009 (the improvements are removing the spurious > > diff hunk, and better comments.

Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-19 Thread Joseph S. Myers
On Tue, 19 Nov 2013, Basile Starynkevitch wrote: > Thanks for your attention. I am attaching a slightly improved patch > against trunk svn rev. 305009 (the improvements are removing the spurious > diff hunk, and better comments.) Still OK in the absence of plugin maintainer objections. -- Jos

Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-19 Thread Diego Novillo
On Tue, Nov 19, 2013 at 11:16 AM, Joseph S. Myers wrote: > On Tue, 19 Nov 2013, Basile Starynkevitch wrote: > >> Thanks for your attention. I am attaching a slightly improved patch >> against trunk svn rev. 305009 (the improvements are removing the spurious >> diff hunk, and better comments.) > >

Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-19 Thread Basile Starynkevitch
On Mon, Nov 18, 2013 at 10:50:10PM +, Joseph S. Myers wrote: > On Mon, 18 Nov 2013, Basile Starynkevitch wrote: > > > @@ -43,6 +44,7 @@ > >TARGET_OPTF. */ > > #include "tm_p.h" /* For C_COMMON_OVERRIDE_OPTIONS. */ > > > > + > > #ifndef DOLLARS_IN_

Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-18 Thread Joseph S. Myers
On Mon, 18 Nov 2013, Basile Starynkevitch wrote: > @@ -43,6 +44,7 @@ > TARGET_OPTF. */ > #include "tm_p.h"/* For C_COMMON_OVERRIDE_OPTIONS. */ > > + > #ifndef DOLLARS_IN_IDENTIFIERS > # define DOLLARS_IN_IDENTIFIERS true > #endif This is a spur

patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-18 Thread Basile Starynkevitch
Hello All After http://gcc.gnu.org/ml/gcc/2013-11/msg00360.html & http://gcc.gnu.org/ml/gcc/2013-11/msg00361.html & http://gcc.gnu.org/ml/gcc/2013-11/msg00347.html I am proposing the following patch ## patch Index: gcc/plugin.def