This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: RFA: shrink main_type


On Mon, Aug 18, 2008 at 05:00:09PM +0400, Joel Brobecker wrote:
>   /* Flags about this type.  This field appears at this location
>      because it packs nicely here.  */
> 
>   ENUM_BITFIELD(type_flag_value) flags : 18;

If you're going to do this anyway, why not make them individual
bitfields?  Would that be too disruptive?

I don't think it's particularly useful to change the type of this to
the enum since we don't put enum values in it, just bitwise
combination of them.  (Isn't that invalid in C++?)

And if you want to repack the upper/lower bounds fields, I bet they
can move into type_specific.

-- 
Daniel Jacobowitz
CodeSourcery


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]