RFR: JDK-8326974: ODR violation in macroAssembler_aarch64.cpp [v2]

Aleksey Shipilev shade at openjdk.org
Thu Feb 29 17:00:53 UTC 2024


On Thu, 29 Feb 2024 16:04:03 GMT, Andrew Haley <aph at openjdk.org> wrote:

>> This is a slightly different patch from the one suggested by the bug reporter. It doesn't make any sense to export those classes globally, so I've given them internal linkage.
>
> Andrew Haley has updated the pull request incrementally with one additional commit since the last revision:
> 
>   JDK-8326974: ODR violation in macroAssembler_aarch64.cpp

Looks fine.

src/hotspot/cpu/aarch64/macroAssembler_aarch64.cpp line 151:

> 149: //       110___10100 (i.e. requires insn[31:21] == 11010010100)
> 150: //
> 151: 

Is this newline addition necessary?

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

Marked as reviewed by shade (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18056#pullrequestreview-1909411315
PR Review Comment: https://git.openjdk.org/jdk/pull/18056#discussion_r1507901707


More information about the hotspot-dev mailing list