FEX-Emu / FEX-Emu/FEX

Consume less memory with relocations

Open
#1,695 0 comments 1 reaction 1 assignee Assigned to @skmp View on GitHub
Dominant language
C++
Stars
8k
Forks
351
Avg merge
12h 31m
Merged PRs (30d)
102

Description

https://github.com/FEX-Emu/FEX/blob/b5ae9e4c976fd2c542502e48c3e734c7bdd0e089/External/FEXCore/Source/Interface/Core/ObjectCache/Relocations.h#L72

Currently relocations use a union which causes them all to be 48 bytes in size.
We can save a bit of allocation overhead by not using a union. Thunks are significantly less common than the other relocations, which means that 16 (or 10 when packed) ends up being quite a bit smaller than 48.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.