On 07/10/2016 09:32 AM, Bernd Edlinger wrote:
On 07/10/16 10:23, Eric Botcazou wrote:
I expect it still has an effect due to the same reason as the
DECL_OFFSET_ALIGN thing - memory reference expansion doesn't gather
information from the full reference but tries to re-cover it from
the pieces returned from get_inner_reference. So I guess we need
to fix that first.
The situation is a bit different though, because the main (and single?) case
for which TYPE_ALIGN_OK had been necessary in the middle-end was made obsolete
by a gigi change at some point. And indeed extensive testing of Bernd's patch
on a strict-alignment platform didn't reveal any issue.
So no objection to the patch on principle by me.
Thank you, Eric.
So following Jeff's suggestion, here is an updated patch that allocates
a new lang_flag_7 out of the spare bits in tree_type_common.
The only difference to the previous version is in tree-core.h where
tree_type_common is declared, and print-tree.c where the new flag
is printed.
Boot-strapped and reg-tested on armv7-linux-gneabihf.
OK for trunk?
OK.
jeff
