RFR(XXS) 6664815: Eliminate redundant memcpy operation in jni_GetStringUTFRegion

Ivan Gerasimov ivan.gerasimov at oracle.com
Fri Apr 4 07:30:49 UTC 2014


Hi Marcus!

While it's not critical and not really connected to the issue you're 
solving, wouldn't it be better to replace

344     p = utf8_write(p, base[index]);

with

344     p = utf8_write(p,*c*);

in src/share/vm/utilities/utf8.cpp?

Sincerely yours,
Ivan


On 03.04.2014 17:17, Marcus Larsson wrote:
> Hello,
>
> I would like reviews for the following bugfix removing a redundant 
> memcpy.
> Built on all platforms and tested with JCK.
>
> Webrev:
> http://cr.openjdk.java.net/~dsimms/6664815/
>
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-6664815
>
> Regards,
> Marcus
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/attachments/20140404/d605ac81/attachment.html>


More information about the hotspot-runtime-dev mailing list