[lworld] RFR: 8263067: [lworld] Use tlab_allocate in MacroAssembler::store_inline_type_fields_to_buf

[Tobias Hartmann]

On Fri, 5 Mar 2021 07:24:34 GMT, Yi Yang <github.com+5010047+kelthuzadx at openjdk.org> wrote:

> TLAB allocation in MacroAssembler::store_inline_type_fields_to_buf looks duplicated with MacroAssembler::tlab_allocate, we might use existing MacroAssembler::tlab_allocate instead of duplicated ones. It will generate safer and faster code. (See details on JBS). All jtreg tests are passed with slowdebug mode.
> 
> Thanks!
> Yang

Hi Yang,

I had this on my To-Do list as well, thanks for fixing! I'm just wondering if we shouldn't also attempt an allocation in eden if `UseTLAB` is disabled? Similar to what code in `MacroAssembler::allocate_instance` does.

Thanks,
Tobias

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

PR: https://git.openjdk.java.net/valhalla/pull/362