On Mon, 3 Jul 2023 06:47:04 GMT, Daniel Jeliński <djelin...@openjdk.org> wrote:
>> src/hotspot/cpu/aarch64/aarch64.ad line 2288:
>>
>>> 2286:
>>> //=============================================================================
>>> 2287:
>>> 2288: const bool Matcher::match_rule_supported(int opcode) {
>>
>> Have to wonder if these were all meant to be `bool Match:xxx() const {`?
>
> Yes, I think that may have been the original intent. I'll add const on these
> functions.
...actually these methods are static, and static functions can't be
const-qualified.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14674#discussion_r1250385599
