[crac] RFR: 8367975: [CRaC] Pattern in CRaCCheckpointTo
Radim Vansa
rvansa at openjdk.org
Tue Sep 23 12:24:36 UTC 2025
On Mon, 22 Sep 2025 14:59:55 GMT, Timofei Pushkin <tpushkin at openjdk.org> wrote:
>> Add support for pattern in `CRaCCheckpointTo` VM option.
>
> src/hotspot/share/runtime/crac.cpp line 188:
>
>> 186: check_no_width();
>> 187: #ifndef ARCHPROPNAME // defined by build scripts
>> 188: # define ARCHPROPNAME "unknown"
>
> Why not assert it is defined?
> Suggestion:
>
> #error ARCHPROPNAME macro must be defined
I had the `define` here mostly to avoid IDE errors, but OK...
-------------
PR Review Comment: https://git.openjdk.org/crac/pull/264#discussion_r2372139319
More information about the crac-dev
mailing list