On Tue, 13 Jul 2021 17:05:25 GMT, Vladimir Ivanov <vliva...@openjdk.org> wrote:
>> Thanks for the suggestion. I'd like to keep it the way it is though, so that >> the assert message contains the `has_last_frame` & `java_call_counter` >> values. (this was one of the reason I did this refactor as well, since the >> assert I was hitting out of those 3 didn't contain that info). > > You can keep the assert inside `verify_frame_info()` which dumps additional > data. (Yes, it's a bit confusing: assert inside an assert :-) ) FTR I'm fine with it either way. ------------- PR: https://git.openjdk.java.net/jdk17/pull/149