-
-
Notifications
You must be signed in to change notification settings - Fork 238
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix some symbols not found in riscv64 #136
base: master
Are you sure you want to change the base?
Conversation
Can you send the |
Okay, it's here. |
I can still find these symbol in your |
Is it okay to use these two symbols?
ENTRY art_quick_to_interpreter_bridge
SETUP_SAVE_REFS_AND_ARGS_FRAME
// uint64_t artQuickToInterpreterBridge(ArtMethod* method, Thread* self, ArtMethod** sp)
// a0 will contain ArtMethod*
mv a1, xSELF
mv a2, sp
call artQuickToInterpreterBridge
// TODO: no need to restore arguments in this case.
RESTORE_SAVE_REFS_AND_ARGS_FRAME
fmv.d.x fa0, a0 // copy the result to FP result register
RETURN_OR_DELIVER_PENDING_EXCEPTION_REG t0
END art_quick_to_interpreter_bridge |
Aren't we using these two symbols? |
extern "C" void art_quick_to_interpreter_bridge(ArtMethod*);
static inline const void* GetQuickToInterpreterBridge() {
return reinterpret_cast<const void*>(art_quick_to_interpreter_bridge);
}
bool ClassLinker::IsQuickToInterpreterBridge(const void* entry_point) const {
return (entry_point == GetQuickToInterpreterBridge()) ||
(quick_to_interpreter_bridge_trampoline_/*nullptr*/ == entry_point);
} |
Oh, I know what you mean. of course, you cannot cast it. If you think it difficult to use artQuickToInterpreterBridge to obtain art_quick_to_interpreter_bridge, maybe we instead use |
Sounds like a good idea |
The symbols

art_quick_generic_jni_trampoline
andart_quick_to_interpreter_bridge
no longer exist in baklava riscv64, so I found another way to lookup the addresses of these two symbols.