On Mon, 2013-08-05 at 17:03 -0400, David Malcolm wrote: > On Mon, 2013-07-29 at 15:41 -0600, Jeff Law wrote: > > On 07/26/2013 09:04 AM, David Malcolm wrote: > > > This patch is the hand-written part of the conversion of passes from > > > C structs to C++ classes. It does not work without the subsequent > > > autogenerated part, which is huge. > > [ ... ] > > With the changes from pipeline -> pass_manager this is fine. As is the > > follow-up autogenerated patch. > > I've committed this and the other patches that needed to go with it to > trunk, with the name fix, having successfully bootstrapped and tested > them on x86_64-unknown-linux-gnu - so opt_pass is now a C++ class > hierarchy, allowing for pass instances to hold pass-specific data > (albeit not GTY refs yet),
Did you follow my suggestion of putting __FILE__ and __LINE__ in instances of opt_pass? I really hope that will go into 4.9! Cheers -- Basile STARYNKEVITCH http://starynkevitch.net/Basile/ email: basile<at>starynkevitch<dot>net mobile: +33 6 8501 2359 8, rue de la Faiencerie, 92340 Bourg La Reine, France *** opinions {are only mine, sont seulement les miennes} ***