RFR: First refactoring (thaw_fast) [v2]

[Frederic Parain]

On Wed, 20 Apr 2022 17:51:26 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> src/hotspot/share/oops/stackChunkOop.inline.hpp line 179:
>> 
>>> 177:   bool empty = false;
>>> 178:   StackChunkFrameStream<ChunkFrames::CompiledOnly> f(this);
>>> 179:   intptr_t* const chunk_sp = start_address() + sp();
>> 
>> This derelativizes the sp() doesn't it?
>
> chunk_sp is only used for asserts? Not that I think it should go into messy DEBUG_ONLY conditionals though.  Maybe these asserts should go into StackChunkFrameStream instead?

Yes, chunk_sp is the derelativized value of sp().
chunk_sp is only used in asserts. Some asserts are specific to the current case of a compiled method.

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

PR: https://git.openjdk.java.net/loom/pull/158