On Tue, 14 Dec 2021 06:57:36 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
> SA had a bunch of vtable related code that searched for the vtable pointer of > a hotspot object in a few places. This was all related to weird ways that > Solaris would store the vtable pointer. In the case of > https://bugs.openjdk.java.net/browse/JDK-8269830, this extra vtable searching > code introduced a bug in BasicTypeDataBase.addressTypeIsEqualToType(). Since > we don't support Solaris anymore, this the bug was fixed by removing this > Solaris support. At the time I fixed JDK-8269830, I noticed that > BasicTypeDataBase.findDynamicTypeForAddress() had similar logic for vtable > searching, although didn't seemed to be causing any bugs at the time. I filed > JDK-8269838 to eventually clean it up, and that's what this PR is doing. This pull request has now been integrated. Changeset: 7517c85d Author: Chris Plummer <cjplum...@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/7517c85da3236e5274580645d4c3cdecb1db25c5 Stats: 36 lines in 1 file changed: 0 ins; 28 del; 8 mod 8269838: BasicTypeDataBase.findDynamicTypeForAddress(addr, basetype) can be simplified Reviewed-by: kevinw, sspitsyn ------------- PR: https://git.openjdk.java.net/jdk/pull/6827