RFR: 8250269: Replace ATTRIBUTE_ALIGNED with alignas [v15]

Julian Waters jwaters at openjdk.org
Wed Apr 19 07:06:49 UTC 2023


On Wed, 12 Apr 2023 01:36:01 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> I've been meaning to review this but have been swamped. Sorry.
> 
> I don't think this change to HotSpot should be combined with JDK-8305341 / PR#13258.
> 
> I'm concerned there might be uses of ATTRIBUTE_ALIGNED in other places than at the front of the declaration (like the fixed offset_of macro in the proposed changes). Obviously there aren't any that break compilation. But is alignas in other places valid but with a different meaning? For a discussion of the kind of thing I'm concerned about, see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108796

I get what you mean, I'll search HotSpot's codebase to see whether this issue pops up in any of our existing code. It's a little tempting to just replace every instance of this macro with alignas, but I guess I'll avoid doing that to keep the changes minimal

-------------

PR Comment: https://git.openjdk.org/jdk/pull/11431#issuecomment-1514230350


More information about the hotspot-runtime-dev mailing list