On Fri, 15 Dec 2023 08:03:45 GMT, Julian Waters <jwat...@openjdk.org> wrote:
>> src/hotspot/os_cpu/linux_riscv/vm_version_linux_riscv.cpp line 77:
>>
>>> 75: #define read_csr(csr) \
>>> 76: ({ \
>>> 77: unsigned long __v; \
>>
>> Can this change be made separately? I'd like to have the C++17 switch be as
>> clean as possible.
>
> No problem!
There are, strangely, many more register keywords in the JDK codebase than just
this one, but none of them throw the same errors, only this one does
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14988#discussion_r1427676677
