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