On Sat, 21 Mar 2020, Tobias Burnus wrote: > On 3/21/20 8:03 AM, Richard Biener wrote: > > >> OK for the trunk? > > It should be TYPE_ALIGN (type). OK with that change. > > I am confused. The patch has: > + SET_DECL_ALIGN (link_ptr_var, TYPE_ALIGN (ptr_type_node)); > which looks correct and to uses already TYPE_ALIGN?!? > > > Note this fails to honor target bits so it might be better to lay out the > > decl properly? > > However, that's a good point. Let's do it properly by calling layout_decl > — indirectly, by calling build_decl. > > OK?
OK. Thanks, Richard.
