> Well, I did that. Apologies for not mentioning that. OK, on which strict-alignment platform did you test it with Ada enabled?
> The change on the ada interface is actually not critical, because all > invocations of get_inner_reference there used keep_aligning == false, as > did the majority of all other invocations. Sure, but that's not the point... > What changes with that patch, is that get_inner_reference(...., true) could > return a VIEW_CONVERT_EXPR, which is now obsolete. ...that's what needs to be properly verified indeed. > If it is causing any trouble, I can revert that change of course. You might want to take a look at PR middle-end/17746, it took about 3 months almost a decade ago to find something plausible to fix this fundamental issue for Ada on strict-aligmnent platforms so I'd rather not go through this again. -- Eric Botcazou
