Skip to content

Commit

Permalink
Allow loader data without local offsets
Browse files Browse the repository at this point in the history
Class chain offsets do not necessarily need to be present when fetching
class loader data from the client in getClassRecord. We only really care
about the name of the loader at this point, since that is what is used
to create the server ClassLoader record. The server already tolerates
not having the local class chain offset cached - if it ever requires
that information, say during compilation, it will request the offset
from the client again.
  • Loading branch information
cjjdespres committed Jan 30, 2024
1 parent 0892da2 commit a2f7da5
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions runtime/compiler/runtime/JITClientSession.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -896,8 +896,7 @@ ClientSessionData::getClassRecord(J9Class *clazz, JITServer::ServerStream *strea
auto recv = stream->read<uintptr_t, std::string>();
uintptr_t offset = std::get<0>(recv);
auto &name = std::get<1>(recv);

if (offset)
if (!name.empty())
{
OMR::CriticalSection cs(getROMMapMonitor());
auto it = getROMClassMap().find((J9Class *)clazz);
Expand Down

0 comments on commit a2f7da5

Please sign in to comment.