[PATCH] D132988: Changes to use a string pool for symbols

[Jared Wyles via Phabricator via llvm-commits]

jaredwy updated this revision to Diff 523650.
jaredwy added a comment.
Herald added subscribers: luke, pcwang-thead, frasercrmck, luismarques, apazos, sameer.abuasal, s.egerton, Jim, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, niosHD, sabuasal, simoncook, johnrusso, rbar, asb.

This further uses symbol ptrs across jitlink, i think in most places now we use it for the symbol name, there is some clean up to go, but this is a good starting point


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132988/new/

https://reviews.llvm.org/D132988

Files:
  llvm/include/llvm/ExecutionEngine/JITLink/COFF.h
  llvm/include/llvm/ExecutionEngine/JITLink/COFF_x86_64.h
  llvm/include/llvm/ExecutionEngine/JITLink/ELF.h
  llvm/include/llvm/ExecutionEngine/JITLink/ELF_aarch32.h
  llvm/include/llvm/ExecutionEngine/JITLink/ELF_aarch64.h
  llvm/include/llvm/ExecutionEngine/JITLink/ELF_i386.h
  llvm/include/llvm/ExecutionEngine/JITLink/ELF_loongarch.h
  llvm/include/llvm/ExecutionEngine/JITLink/ELF_riscv.h
  llvm/include/llvm/ExecutionEngine/JITLink/ELF_x86_64.h
  llvm/include/llvm/ExecutionEngine/JITLink/JITLink.h
  llvm/include/llvm/ExecutionEngine/JITLink/MachO.h
  llvm/include/llvm/ExecutionEngine/JITLink/MachO_arm64.h
  llvm/include/llvm/ExecutionEngine/JITLink/MachO_x86_64.h
  llvm/include/llvm/ExecutionEngine/JITLink/TableManager.h
  llvm/include/llvm/ExecutionEngine/Orc/Layer.h
  llvm/lib/ExecutionEngine/JITLink/COFF.cpp
  llvm/lib/ExecutionEngine/JITLink/COFFLinkGraphBuilder.cpp
  llvm/lib/ExecutionEngine/JITLink/COFFLinkGraphBuilder.h
  llvm/lib/ExecutionEngine/JITLink/COFF_x86_64.cpp
  llvm/lib/ExecutionEngine/JITLink/EHFrameSupport.cpp
  llvm/lib/ExecutionEngine/JITLink/ELF.cpp
  llvm/lib/ExecutionEngine/JITLink/ELFLinkGraphBuilder.h
  llvm/lib/ExecutionEngine/JITLink/ELF_aarch32.cpp
  llvm/lib/ExecutionEngine/JITLink/ELF_aarch64.cpp
  llvm/lib/ExecutionEngine/JITLink/ELF_i386.cpp
  llvm/lib/ExecutionEngine/JITLink/ELF_loongarch.cpp
  llvm/lib/ExecutionEngine/JITLink/ELF_riscv.cpp
  llvm/lib/ExecutionEngine/JITLink/ELF_x86_64.cpp
  llvm/lib/ExecutionEngine/JITLink/JITLink.cpp
  llvm/lib/ExecutionEngine/JITLink/JITLinkGeneric.cpp
  llvm/lib/ExecutionEngine/JITLink/JITLinkMemoryManager.cpp
  llvm/lib/ExecutionEngine/JITLink/MachO.cpp
  llvm/lib/ExecutionEngine/JITLink/MachOLinkGraphBuilder.cpp
  llvm/lib/ExecutionEngine/JITLink/MachOLinkGraphBuilder.h
  llvm/lib/ExecutionEngine/JITLink/MachO_arm64.cpp
  llvm/lib/ExecutionEngine/JITLink/MachO_x86_64.cpp
  llvm/lib/ExecutionEngine/JITLink/PerGraphGOTAndPLTStubsBuilder.h
  llvm/lib/ExecutionEngine/Orc/COFFPlatform.cpp
  llvm/lib/ExecutionEngine/Orc/ELFNixPlatform.cpp
  llvm/lib/ExecutionEngine/Orc/ExecutionUtils.cpp
  llvm/lib/ExecutionEngine/Orc/IndirectionUtils.cpp
  llvm/lib/ExecutionEngine/Orc/MachOPlatform.cpp
  llvm/lib/ExecutionEngine/Orc/ObjectFileInterface.cpp
  llvm/lib/ExecutionEngine/Orc/ObjectLinkingLayer.cpp
  llvm/tools/llvm-jitlink/llvm-jitlink-coff.cpp
  llvm/tools/llvm-jitlink/llvm-jitlink-elf.cpp
  llvm/tools/llvm-jitlink/llvm-jitlink-macho.cpp
  llvm/tools/llvm-jitlink/llvm-jitlink.cpp
  llvm/unittests/ExecutionEngine/JITLink/EHFrameSupportTests.cpp
  llvm/unittests/ExecutionEngine/JITLink/LinkGraphTests.cpp
  llvm/unittests/ExecutionEngine/Orc/ObjectLinkingLayerTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D132988.523650.patch
Type: text/x-patch
Size: 105070 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230519/80a05b9c/attachment-0001.bin>