Re: [PATCH 06/11] Rewrite how instances of passes are cloned

2013-08-01 Thread Richard Henderson
On 08/01/2013 07:55 AM, David Malcolm wrote: > On Fri, 2013-07-26 at 11:04 -0400, David Malcolm wrote: >> > gcc/ >> > >> >Rewrite how instances of passes are cloned to remove assumptions >> >about their sizes (thus allowing pass subclasses to have >> >additional data fields, albeit non

Re: [PATCH 06/11] Rewrite how instances of passes are cloned

2013-08-01 Thread David Malcolm
On Thu, 2013-08-01 at 13:55 -0400, David Malcolm wrote: [...snip...] > OK for trunk? (on top of the other patches, of course; see notes in > http://gcc.gnu.org/ml/gcc-patches/2013-08/msg00041.html in how I've > tested this). ...with "pipeline" renamed to "pass_manager", of course.

Re: [PATCH 06/11] Rewrite how instances of passes are cloned

2013-08-01 Thread David Malcolm
This patch does more than just remove the hardcoded assumptions about pass sizes - as noted in http://gcc.gnu.org/ml/gcc-patches/2013-08/msg00041.html it also is needed to ensure that per-pass dumpfiles get the correct switch names. So the short version is that patch 6 is needed for patches 3-5 to

[PATCH 06/11] Rewrite how instances of passes are cloned

2013-07-26 Thread David Malcolm
gcc/ Rewrite how instances of passes are cloned to remove assumptions about their sizes (thus allowing pass subclasses to have additional data fields, albeit non-GC-managed ones at this point). * passes.c (make_pass_instance): Now that passes have clone met