On Mon, 9 Dec 2024 21:35:42 GMT, ExE Boss <d...@openjdk.org> wrote: >> yes. it is needed. > > This is **C++**, so yes. > Suggestion: > > macro(_modifiers_offset, k, vmSymbols::modifiers_name(), > int_signature, false);
I see, there's a trailing semi somewhere in the expansion of this macro so it compiles, but I added one in. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22652#discussion_r1878263513
