On Thu, 13 Feb 2025 17:14:59 GMT, Vladimir Kozlov <k...@openjdk.org> wrote:
>> Remove virtual methods from CodeBlob and nmethod to simplify >> saving/restoring in Leyden AOT cache. It avoids the need to patch hidden >> VPTR pointer to class's virtual table. >> >> Added C++ static asserts to make sure no virtual methods are added in a >> future. >> >> Fixed/cleaned SA code which process CodeBlob and its subclasses. Use >> `CodeBlob::_kind` field value to determine the type of blob. >> >> Tested tier1-5, hs-tier6-rt (for JFR testing), stress, xcomp > > Vladimir Kozlov has updated the pull request incrementally with one > additional commit since the last revision: > > rename SA argument Saying that, I agree that I need to add comments explaining printing API and how Vptr class will work. I will work on @xmas92 suggestions and look on using `_impl`. I will try to look on templates @rose00 suggested but I don't want to complicate code for just for few print methods. ------------- PR Comment: https://git.openjdk.org/jdk/pull/23533#issuecomment-2657303967
