Cleanup method inspection callbacks from jvmti
We used to use three different inspection callbacks for kind of similar
checks.
- IsMethodBeingInspected - returns true if method has breakpoints or if
we need to observe any locals (ideally for only this method but
current implementation is not ideal). This is used to check if we can
JIT code.
- IsMethodSafeToJit - returns true if method has breakpoints. Though we
also check if a method is being inspected making this check redundant.
- IsMethodNeedsDebugVersion - this is used to check if we need to use
switch interpreter. This was always returning true forcing us to use
switch interpreter when debugger is attached. We should use
IsMethodBeingInspected instead.
This CL merges all these into IsMethodBeingInspected callback.
Bug: 206029744
Test: test.py --gtest --run-test
Change-Id: Idcde354f39775092be5ddb79f09a92f81e07b1ee
7 files changed