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

[Bertrand Delsart]

Hi Tom,

New webrev with the one line move you suggested and some changes in the 
comments:
http://cr.openjdk.java.net/~bdelsart/7025485/webrev.01/

Comments were mainly to help the review. Now that it is done, since you 
prefer the history not to be kept in the files, I've removed all the 
history part of the old comment and only kept the first sentence, which 
explains what it does (adding the "On some platforms"):
+  // On some platforms, we need a way to pass fp to the unpacking code
+  // so the skeletal frames come out correct.

If you'd like to use this push to complete the information, feel free to 
suggest additional comments that I will insert before pushing. Else, 
please confirm that I have your approval for the push.

Thanks,

Bertrand.

On 03/ 9/11 06:42 PM, Tom Rodriguez wrote:
> 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.
>
> tom
>
> 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:
>>
>> http://cr.openjdk.java.net/~bdelsart/7025485/webrev.00/
>>
>> 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 oracle.com
>> 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.
>> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>


-- 
Bertrand Delsart,                         bertrand.delsart at oracle.com
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.
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~