On Tue, 31 Oct 2023 14:30:26 GMT, Robbin Ehn <[email protected]> wrote:
>> src/utils/hsdis/binutils/hsdis-binutils.c line 340:
>>
>>> 338: native_bfd,
>>> 339: /* On some archs we get warnings, if
>>> we pass empty options */
>>> 340: (app_data->insn_options[0] == '\0') ?
>>> NULL : app_data->insn_options);
>>
>> What if `caller_options == NULL`?
>
> The parse_caller_options handles the NULL case, so I forgot about the early
> bailout.
>
> for (p = caller_options; p != NULL; ) {
> }
> *iop = '\0';
>
>
> Sorry.
Just remove the if, and always call parse_caller_options ?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16433#discussion_r1377685996
