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

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

Roland Westrelin wrote:

Isn't fpruntimefallback in assemblerx86.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.

sharedRuntimex8632.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