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

Yi Yang github.com+5010047+kelthuzadx at openjdk.java.net
Mon Mar 8 03:49:30 UTC 2021


> 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

Yi Yang has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:

  improve buffer instance allocation in MacroAssembler::store_inline_type_fields_to_buf

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

Changes:
  - all: https://git.openjdk.java.net/valhalla/pull/362/files
  - new: https://git.openjdk.java.net/valhalla/pull/362/files/e4b62d19..90e3b7c9

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=valhalla&pr=362&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=valhalla&pr=362&range=00-01

  Stats: 46 lines in 2 files changed: 17 ins; 9 del; 20 mod
  Patch: https://git.openjdk.java.net/valhalla/pull/362.diff
  Fetch: git fetch https://git.openjdk.java.net/valhalla pull/362/head:pull/362

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



More information about the valhalla-dev mailing list