RFR: 8278518: String(byte[], int, int, Charset) constructor and String.translateEscapes() miss bounds check elimination [v2]

Roland Westrelin roland at openjdk.java.net
Wed Jan 12 10:22:56 UTC 2022


On Tue, 11 Jan 2022 18:34:18 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Roland Westrelin has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review
>
> src/hotspot/share/ci/ciTypeFlow.cpp line 2541:
> 
>> 2539:         return leaf; // Already in list
>> 2540:       if (current->head()->pre_order() < lp_pre_order)
>> 2541:         break;
> 
> Can you add `{}` for these checks too.

fixed in new commit

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

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


More information about the hotspot-compiler-dev mailing list