Hi!

On Thu, Sep 13, 2018 at 04:41:25PM +0300, Ville Voutilainen wrote:
> --- a/gcc/cp/cp-tree.h
> +++ b/gcc/cp/cp-tree.h
> @@ -2046,7 +2046,7 @@ struct GTY(()) lang_type {
>    unsigned lazy_copy_assign : 1;
>    unsigned lazy_destructor : 1;
>    unsigned has_const_copy_ctor : 1;
> -  unsigned has_complex_copy_ctor : 1;
> +  unsigned has_complex_copy_ctor : 2;
>    unsigned has_complex_copy_assign : 1;
>  
>    unsigned non_aggregate : 1;

Just a formatting nit, there is currently an empty new-line after each group
of 8 bits, so if you add one bit in the middle, the empty lines need to move
as well (i.e. empty line before has_complex_copy_assign instead of after it
and ditto for has_complex_move_ctor).

I'll defer actual review to Jason/Nathan.

> @@ -2070,7 +2070,7 @@ struct GTY(()) lang_type {
>    /* There are some bits left to fill out a 32-bit word.  Keep track
>       of this by updating the size of this bitfield whenever you add or
>       remove a flag.  */
> -  unsigned dummy : 4;
> +  unsigned dummy : 3;
>  
>    tree primary_base;
>    vec<tree_pair_s, va_gc> *vcall_indices;

        Jakub

Reply via email to