[PATCH]: Remove #ifdef PCC_BITFIELD_TYPE_MATTERS check from dwarf2out.c

[PATCH]: Remove #ifdef PCC_BITFIELD_TYPE_MATTERS check from dwarf2out.c

[Uros Bizjak]

[-- Attachment #1: Type: text/plain, Size: 320 bytes --]

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.

[-- Attachment #2: p.diff.txt --]
[-- Type: text/plain, Size: 804 bytes --]

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)

Re: [PATCH]: Remove #ifdef PCC_BITFIELD_TYPE_MATTERS check from dwarf2out.c

[Jeff Law]

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