RFR(S): 8148752: Compiled StringBuilder code throws StringIndexOutOfBoundsException
vladimir.x.ivanov at oracle.com
Tue Feb 9 12:56:54 UTC 2016
If possible, please, provide a test case.
PS: the bug is there from day #0. I am surprised it hasn't been
discovered for so long...
On 2/9/16 3:01 PM, Tobias Hartmann wrote:
> please review the following patch.
> GraphBuilder::try_method_handle_inline() handles long and double arguments incorrectly when casting them to their actual type. Long/double arguments occupy two slots (see Parse::do_get_xxx() -> push_pair()). Currently, these two slots are treated as two individual arguments, causing incorrect casting of arguments.
> In this case, an Object argument is casted to String and therefore being used as a String (although it's actually of type 'ResolvedJavaMethod'). We later fail by throwing exceptions in compiled code. For a detailed evaluation see my comments in the bug .
> We should handle this like C1 does in GraphBuilder::try_method_handle_inline().
>  https://bugs.openjdk.java.net/browse/JDK-8148752?focusedCommentId=13896557&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13896557
More information about the hotspot-compiler-dev