This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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: sh64-elf (SH5) port

Alexandre Oliva wrote:-

> +
> +  /* Return true if bitfields in RECORD_TYPE should follow the
> +     Microsoft Visual C++ bitfield layout rules.  */
> +  bool (* ms_bitfield_layout_p) PARAMS ((tree record_type));

Why not just have a bool variable?  This would be clearer and
more efficient.  It appears that

1) None of the implementations of this function use the argument.
2) All the implementations know the answer at GCC build time.

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