Skip to content

Commit 53c8efc

Browse files
jbachorikclaude
andcommitted
revert: undo review changes to pre-existing code
- dwarf.h: restore u32 cfa (was already u32 on main; int was wrong) - vmStructs.cpp: remove isThreadAccessible() guard from getExecutionMode/getOSThreadState (pre-existing code, not part of this branch) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent ef42cd7 commit 53c8efc

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

ddprof-lib/src/main/cpp/dwarf.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ const int INITIAL_PC_OFFSET = DW_LINK_REGISTER;
7474

7575
struct FrameDesc {
7676
u32 loc;
77-
int cfa;
77+
u32 cfa;
7878
int fp_off;
7979
int pc_off;
8080

ddprof-lib/src/main/cpp/hotspot/vmStructs.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -696,7 +696,7 @@ ExecutionMode VMThread::getExecutionMode() {
696696

697697
VMThread* vm_thread = VMThread::current();
698698
// Not a JVM thread - native thread, e.g. thread launched by JNI code
699-
if (vm_thread == nullptr || !vm_thread->isThreadAccessible()) {
699+
if (vm_thread == nullptr) {
700700
return ExecutionMode::NATIVE;
701701
}
702702

@@ -722,7 +722,7 @@ ExecutionMode VMThread::getExecutionMode() {
722722

723723
OSThreadState VMThread::getOSThreadState() {
724724
VMThread* vm_thread = VMThread::current();
725-
if (vm_thread == nullptr || !vm_thread->isThreadAccessible()) {
725+
if (vm_thread == nullptr) {
726726
return OSThreadState::UNKNOWN;
727727
}
728728
int raw_thread_state = vm_thread->state();

0 commit comments

Comments
 (0)