diff options
| author | 2020-08-31 15:03:20 +0100 | |
|---|---|---|
| committer | 2020-09-07 07:29:59 +0000 | |
| commit | 4717175e40a19e79af904dfb7b7dd13f046debd7 (patch) | |
| tree | 426f040eacf5a8305f8bb8d504bd509824d984c8 /openjdkjvmti/ti_method.cc | |
| parent | 1faacf59b9f74e9d2de4e0331ef0cdfcf132225d (diff) | |
Move code item to the data pointer and remove code_item_offset.
This saves 4 bytes on 32bit and 8 bytes on 64bit on ArtMethod.
Also update nterp to directly fetch the code item from the data pointer.
Test: test.py
Bug: 112676029
Change-Id: Ic01f43c7ccf2cbce1ec517478e81362232d36371
Diffstat (limited to 'openjdkjvmti/ti_method.cc')
| -rw-r--r-- | openjdkjvmti/ti_method.cc | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/openjdkjvmti/ti_method.cc b/openjdkjvmti/ti_method.cc index e7f071fac8..f2646c6c45 100644 --- a/openjdkjvmti/ti_method.cc +++ b/openjdkjvmti/ti_method.cc @@ -190,7 +190,8 @@ jvmtiError MethodUtil::GetArgumentsSize(jvmtiEnv* env ATTRIBUTE_UNUSED, return ERR(NONE); } - DCHECK_NE(art_method->GetCodeItemOffset(), 0u); + DCHECK(art_method->HasCodeItem()); + DCHECK_NE(art_method->GetCodeItem(), nullptr); *size_ptr = art_method->DexInstructionData().InsSize(); return ERR(NONE); @@ -306,7 +307,8 @@ jvmtiError MethodUtil::GetMaxLocals(jvmtiEnv* env ATTRIBUTE_UNUSED, return ERR(NONE); } - DCHECK_NE(art_method->GetCodeItemOffset(), 0u); + DCHECK(art_method->HasCodeItem()); + DCHECK_NE(art_method->GetCodeItem(), nullptr); *max_ptr = art_method->DexInstructionData().RegistersSize(); return ERR(NONE); @@ -420,7 +422,8 @@ jvmtiError MethodUtil::GetMethodLocation(jvmtiEnv* env ATTRIBUTE_UNUSED, return ERR(NONE); } - DCHECK_NE(art_method->GetCodeItemOffset(), 0u); + DCHECK(art_method->HasCodeItem()); + DCHECK_NE(art_method->GetCodeItem(), nullptr); *start_location_ptr = 0; *end_location_ptr = art_method->DexInstructions().InsnsSizeInCodeUnits() - 1; |