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

Yi Yang github.com+5010047+kelthuzadx at openjdk.java.net
Fri Mar 5 07:29:05 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

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

Commit messages:
 - reuse tlab_allocate in store_inline_type_fields_to_buf

Changes: https://git.openjdk.java.net/valhalla/pull/362/files
 Webrev: https://webrevs.openjdk.java.net/?repo=valhalla&pr=362&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8263067
  Stats: 10 lines in 1 file changed: 1 ins; 6 del; 3 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