RFR: 8272778: Consolidate is_instance and is_instance_inlined in java_lang_String

Coleen Phillimore coleenp at openjdk.java.net
Fri Aug 20 15:24:24 UTC 2021


On Fri, 20 Aug 2021 13:31:02 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

> Simple change of merging `is_instance` and `is_instance_inlined`.
> 
> Test: build

src/hotspot/share/classfile/javaClasses.inline.hpp line 126:

> 124: }
> 125: 
> 126: bool java_lang_String::is_instance(oop obj) {

I don't really see anything wrong with this.  is_instance() is called above in asserts but that shouldn't affect performance by increasing the size of generated code.

-------------

PR: https://git.openjdk.java.net/jdk/pull/5201


More information about the shenandoah-dev mailing list