Request for reviews (L): 7196199: java/text/Bidi/Bug6665028.java failed: Bidi run count incorrect

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Sep 14 10:19:29 PDT 2012


Roland Westrelin wrote:
> Isn't fp_runtime_fallback in assembler_x86.cpp only called from c1/interpreter. In that case does it need to be changed?

I can't find the way to assert there to make sure it is NOT called from C2 code 
generation (which may happen in a future). Any suggestions are welcome.

> 
> sharedRuntime_x86_32.cpp: there's a typo line 72 // to aligh

Fixed.

> 
> Otherwise, it looks good to me.

Thanks,
Vladimir

> 
> Roland.
> 


More information about the hotspot-compiler-dev mailing list