diff mbox series

: Remove #ifdef PCC_BITFIELD_TYPE_MATTERS check from dwarf2out.c

Message ID CAFULd4Y+EO0s119iLUrT84gqdNRspnmLYG7HYAnyqHm5V_C=Ww@mail.gmail.com
State New
Headers show
Series : Remove #ifdef PCC_BITFIELD_TYPE_MATTERS check from dwarf2out.c | expand

Commit Message

Uros Bizjak Dec. 5, 2018, 8:56 p.m. UTC 
This macro is always defined through defaults.h. We can remove #ifdef
check from dwarf2out.c

2018-12-05  Uros Bizjak  <ubizjak@gmail.com>

    * dwarf2out.c (field_byte_offset): Remove
    #ifdef PCC_BITFIELD_TYPE_MATTERS check.

Bootstrapped and regression tested on x86_64-linux-gnu {,-m32}.

OK for mainline?

Uros.

Comments

Jeff Law Dec. 5, 2018, 11:44 p.m. UTC | #1 
On 12/5/18 1:56 PM, Uros Bizjak wrote:
> This macro is always defined through defaults.h. We can remove #ifdef
> check from dwarf2out.c
> 
> 2018-12-05  Uros Bizjak  <ubizjak@gmail.com>
> 
>     * dwarf2out.c (field_byte_offset): Remove
>     #ifdef PCC_BITFIELD_TYPE_MATTERS check.
> 
> Bootstrapped and regression tested on x86_64-linux-gnu {,-m32}.
> 
> OK for mainline?
OK
jeff
diff mbox series

Patch

Index: dwarf2out.c
===================================================================
--- dwarf2out.c	(revision 266788)
+++ dwarf2out.c	(working copy)
@@ -18985,7 +18985,6 @@  field_byte_offset (const_tree decl, struct vlr_con
   if (TREE_CODE (DECL_FIELD_BIT_OFFSET (decl)) != INTEGER_CST)
     return NULL;
 
-#ifdef PCC_BITFIELD_TYPE_MATTERS
   /* We used to handle only constant offsets in all cases.  Now, we handle
      properly dynamic byte offsets only when PCC bitfield type doesn't
      matter.  */
@@ -19100,7 +19099,6 @@  field_byte_offset (const_tree decl, struct vlr_con
       tree_result = wide_int_to_tree (sizetype, object_offset_in_bytes);
     }
   else
-#endif /* PCC_BITFIELD_TYPE_MATTERS */
     tree_result = byte_position (decl);
 
   if (ctx->variant_part_offset != NULL_TREE)