RFR: 8256754: Deoptimization::revoke_for_object_deoptimization: stack processing start call is redundant [v2]

Richard Reingruber rrich at openjdk.java.net
Tue Nov 24 11:31:56 UTC 2020


On Mon, 23 Nov 2020 22:41:03 GMT, Dean Long <dlong at openjdk.org> wrote:

>> Richard Reingruber has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Assert stack is kept gc processed.
>
> The justification for removing this code is that all callers use KeepStackGCProcessedMark.  Is there an assert you can add in place of the removed code that checks this invariant?

Thanks for having a look @dean-long 
> 
> 
> The justification for removing this code is that all callers use KeepStackGCProcessedMark. Is there an assert you can add in place of the removed code that checks this invariant?

I don't think there is an assert I could use. I added code that allows for asserting this.  Let me know which version you prefer.

-------------

PR: https://git.openjdk.java.net/jdk/pull/1381


More information about the hotspot-dev mailing list