On Tue, Apr 28, 2020 at 05:47:27PM +0200, Richard Biener wrote:
> On April 28, 2020 4:04:58 PM GMT+02:00, Jakub Jelinek via Gcc-patches 
> <gcc-patches@gcc.gnu.org> wrote:
> >Hi!
> >
> >On Tue, Apr 28, 2020 at 08:53:31AM -0400, Jason Merrill wrote:
> >> That sounds good.
> >
> >So like this?  Or better name for the new macro?
> 
> I think you miss a hunk for lto/ to compare the flag for tree merging. 

You're right,
        * lto-common.c (compare_tree_sccs_1): Handle DECL_FIELD_ABI_IGNORED.

--- gcc/lto/lto-common.c.jj     2020-04-17 14:18:44.357438048 +0200
+++ gcc/lto/lto-common.c        2020-04-28 17:53:04.809821198 +0200
@@ -1179,6 +1179,7 @@ compare_tree_sccs_1 (tree t1, tree t2, t
          compare_values (DECL_PACKED);
          compare_values (DECL_NONADDRESSABLE_P);
          compare_values (DECL_PADDING_P);
+         compare_values (DECL_FIELD_ABI_IGNORED);
          compare_values (DECL_OFFSET_ALIGN);
        }
       else if (code == VAR_DECL)

added to my copy of the patch.

        Jakub

Reply via email to