On Tue, 31 Oct 2023 13:05:44 GMT, Robbin Ehn <r...@openjdk.org> wrote:
> Hi, please consider. > > insn_options[0] is set to empty string if there is no options (NULL or empty > strings). > Checking it for empty string should cover both cases, caller option is NULL > or caller option is empty string. > > Tested hsdis no longer gives me the warning. 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`? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16433#discussion_r1377661214
