RFR: 8286876: NMT.test_unaliged_block_address_vm_assert fails if using clang toolchain
Johan Sjölen
jsjolen at openjdk.org
Fri Dec 2 09:29:55 UTC 2022
On Fri, 2 Dec 2022 07:09:54 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
>I opened https://bugs.openjdk.org/browse/JDK-8298003 to track the printout error separately and will file a PR for this shortly. I'd like to fix this quickly before JDK 20 code freeze.
I see that you got your reviews, so that's good. I'll fix my version by merging on top of yours and doing any necessary adjustments.
>Is it allowed to do that? This surprises me. We use is_aligned(this) in other places too, e.g. in oopStorage. So maybe this needs a more general solution. @kimbarrett?
All I can find is that C++ talks about objects having an "alignment requirement", I take that to mean that the compiler can assume that alignment for something like `this`.
-------------
PR: https://git.openjdk.org/jdk/pull/11465
More information about the hotspot-runtime-dev
mailing list