Skip to content

Commit

Permalink
Rename TR_InlinedSiteHastTableEntry to TR_InlinedSiteHashTableEntry
Browse files Browse the repository at this point in the history
The identifier "TR_InlinedSiteHashTableEntry" was mistakenly misspelled
as "TR_InlinedSiteHastTableEntry" in various parts of codebase.
This typographical error caused confusion and inconsistencies
throughout the project.
This commit addresses the issue by correcting the typo and renaming the
variable to "TR_InlinedSiteHashTableEntry" consistently throughout the
project.
  • Loading branch information
prajwalbandak committed Jul 27, 2023
1 parent ffcf274 commit ba5f46b
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
10 changes: 5 additions & 5 deletions runtime/compiler/codegen/J9CodeGenerator.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2735,7 +2735,7 @@ static TR_ExternalRelocationTargetKind getReloKindFromGuardSite(TR::CodeGenerato
return type;
}

static void processAOTGuardSites(TR::CodeGenerator *cg, uint32_t inlinedCallSize, TR_InlinedSiteHastTableEntry *orderedInlinedSiteListTable)
static void processAOTGuardSites(TR::CodeGenerator *cg, uint32_t inlinedCallSize, TR_InlinedSiteHashTableEntry *orderedInlinedSiteListTable)
{
TR::list<TR_AOTGuardSite*> *aotGuardSites = cg->comp()->getAOTGuardPatchSites();
for(auto it = aotGuardSites->begin(); it != aotGuardSites->end(); ++it)
Expand Down Expand Up @@ -2845,7 +2845,7 @@ static void addInlinedSiteRelocation(TR::CodeGenerator *cg,
NULL);
}

static void addInliningTableRelocations(TR::CodeGenerator *cg, uint32_t inlinedCallSize, TR_InlinedSiteHastTableEntry *orderedInlinedSiteListTable)
static void addInliningTableRelocations(TR::CodeGenerator *cg, uint32_t inlinedCallSize, TR_InlinedSiteHashTableEntry *orderedInlinedSiteListTable)
{
// If have inlined calls, now add the relocation records in descending order
// of inlined site index (at relocation time, the order is reverse)
Expand Down Expand Up @@ -2891,11 +2891,11 @@ J9::CodeGenerator::processRelocations()
uint32_t inlinedCallSize = self()->comp()->getNumInlinedCallSites();

// Create temporary hashtable for ordering AOT guard relocations
TR_InlinedSiteHastTableEntry *orderedInlinedSiteListTable = NULL;
TR_InlinedSiteHashTableEntry *orderedInlinedSiteListTable = NULL;
if (inlinedCallSize > 0)
{
orderedInlinedSiteListTable= (TR_InlinedSiteHastTableEntry*)self()->comp()->trMemory()->allocateMemory(sizeof(TR_InlinedSiteHastTableEntry) * inlinedCallSize, heapAlloc);
memset(orderedInlinedSiteListTable, 0, sizeof(TR_InlinedSiteHastTableEntry)*inlinedCallSize);
orderedInlinedSiteListTable= (TR_InlinedSiteHashTableEntry*)self()->comp()->trMemory()->allocateMemory(sizeof(TR_InlinedSiteHashTableEntry) * inlinedCallSize, heapAlloc);
memset(orderedInlinedSiteListTable, 0, sizeof(TR_InlinedSiteHashTableEntry)*inlinedCallSize);
}

// Traverse list of AOT-specific guards and create relocation records
Expand Down
4 changes: 2 additions & 2 deletions runtime/compiler/runtime/J9Runtime.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -115,11 +115,11 @@ typedef struct TR_InlinedSiteLinkedListEntry
} TR_InlinedSiteLinkedListEntry;


typedef struct TR_InlinedSiteHastTableEntry
typedef struct TR_InlinedSiteHashTableEntry
{
TR_InlinedSiteLinkedListEntry *first;
TR_InlinedSiteLinkedListEntry *last;
} TR_InlinedSiteHastTableEntry;
} TR_InlinedSiteHashTableEntry;


typedef enum
Expand Down

0 comments on commit ba5f46b

Please sign in to comment.