diff options
| author | 2018-06-23 22:05:56 +0100 | |
|---|---|---|
| committer | 2018-07-27 15:03:06 +0100 | |
| commit | 8808756b8fba036a9c73a45c800a56be09872364 (patch) | |
| tree | 9b28f314ff516d1d9b85d07f71bdc77fad465e48 /runtime/jit/jit_code_cache.cc | |
| parent | dda4e8b5f2ffed24a2dcdb99d2301d9fc0eb91e2 (diff) | |
Remove frame info from OatQuickMethodHeader.
The information has been moved to CodeInfo,
where it is stored in much more compact way.
The old CL which added the data to CodeInfo cost 0.7%.
This CL saves 2.5% of .oat file size so a win overall.
Test: test-art-host-gtest
Change-Id: I07fcf6f2776c96218f995ba3b57a1e6ccbf5e317
Diffstat (limited to 'runtime/jit/jit_code_cache.cc')
| -rw-r--r-- | runtime/jit/jit_code_cache.cc | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/runtime/jit/jit_code_cache.cc b/runtime/jit/jit_code_cache.cc index 17561d2864..9aa05561f0 100644 --- a/runtime/jit/jit_code_cache.cc +++ b/runtime/jit/jit_code_cache.cc @@ -413,9 +413,6 @@ uint8_t* JitCodeCache::CommitCode(Thread* self, uint8_t* stack_map, uint8_t* method_info, uint8_t* roots_data, - size_t frame_size_in_bytes, - size_t core_spill_mask, - size_t fp_spill_mask, const uint8_t* code, size_t code_size, size_t data_size, @@ -428,9 +425,6 @@ uint8_t* JitCodeCache::CommitCode(Thread* self, stack_map, method_info, roots_data, - frame_size_in_bytes, - core_spill_mask, - fp_spill_mask, code, code_size, data_size, @@ -446,9 +440,6 @@ uint8_t* JitCodeCache::CommitCode(Thread* self, stack_map, method_info, roots_data, - frame_size_in_bytes, - core_spill_mask, - fp_spill_mask, code, code_size, data_size, @@ -759,9 +750,6 @@ uint8_t* JitCodeCache::CommitCodeInternal(Thread* self, uint8_t* stack_map, uint8_t* method_info, uint8_t* roots_data, - size_t frame_size_in_bytes, - size_t core_spill_mask, - size_t fp_spill_mask, const uint8_t* code, size_t code_size, size_t data_size, @@ -796,9 +784,6 @@ uint8_t* JitCodeCache::CommitCodeInternal(Thread* self, new (method_header) OatQuickMethodHeader( (stack_map != nullptr) ? code_ptr - stack_map : 0u, (method_info != nullptr) ? code_ptr - method_info : 0u, - frame_size_in_bytes, - core_spill_mask, - fp_spill_mask, code_size); // Flush caches before we remove write permission because some ARMv8 Qualcomm kernels may // trigger a segfault if a page fault occurs when requesting a cache maintenance operation. |