On Mon, 14 Nov 2022 16:53:07 GMT, Lutz Schmidt <[email protected]> wrote:
>> Xue-Lei Andrew Fan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> include missing os head file > > src/hotspot/share/adlc/output_c.cpp line 536: > >> 534: int printed = snprintf(args, 37, "0x%x, 0x%x, %u", >> 535: resources_used, resources_used_exclusively, element_count); >> 536: assert(printed <= 36, "overflow"); > > if snprintf works correctly (we rely on that), this assert will never fire. Good point. I removed the assert in the new commit. ------------- PR: https://git.openjdk.org/jdk/pull/11115
