Request for Review (XS): leverage shared x86-only deoptimization code

Tom Rodriguez tom.rodriguez at
Wed Mar 9 09:42:58 PST 2011

You should move the initial_fp declaration up into the main section since it isn't an x86 temp anymore.  I know this is a bit nit-picky, but I'd also prefer if the comments you updated just described what was being captured and why it might be needed instead of talking about the history.


On Mar 9, 2011, at 5:40 AM, Bertrand Delsart wrote:

> Hi all,
> Here is a simple webrev to allow other platforms to benefit from
> shared code that was conditionally compiled only on x86 machines:
> This code had initially be modified by explicitly adding to the ifdef the platforms that required it. However, since the code is supposed to be harmless, it is cleaner and better in the long term to activate it by default for all future platforms.
> Bertrand.
> -- 
> Bertrand Delsart,                         bertrand.delsart at
> Sun-Oracle,         180 av. de l'Europe,         ZIRST de Montbonnot,
> 38334 Saint Ismier,                                            FRANCE
> Phone : +33 4 76 18 81 23 Fax : +33 4 76 18 88 88
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> NOTICE: This email message is for the sole use of the intended
> recipient(s) and may contain confidential and privileged
> information. Any unauthorized review, use, disclosure or distribution
> is prohibited. If you are not the intended recipient, please contact
> the sender by reply email and destroy all copies of the original
> message.
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

More information about the hotspot-runtime-dev mailing list