[GOLD] powerpc linker generated .eh_frame
Alan Modra
amodra@gmail.com
Wed Feb 27 22:42:00 GMT 2013
On Wed, Feb 27, 2013 at 06:22:34AM -0800, Ian Lance Taylor wrote:
> > + ~Post_fde();
>
> I don't see where this is defined. I also don't see why the default
> destructor isn't fine. Can you just omit this line?
Ick, I'm not sure how that got there.
> > + Fde* fde_;
> > + section_offset_type cie_offset_;
> > + unsigned char fde_encoding_;
> > +};
>
> Make the class member variables private. Or make this a struct, in
> which case remove the trailing underscores from the field names.
I've made it a struct.
--
Alan Modra
Australia Development Lab, IBM
More information about the Binutils
mailing list