[PATCH] D133994: [BOLT] Change base class of ExecutableFileMemoryManager

Maksim Panchenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 15 16:56:18 PDT 2022


maksfb created this revision.
maksfb added reviewers: yota9, Amir, ayermolo, rafauler.
Herald added a subscriber: treapster.
Herald added a project: All.
maksfb requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

When we derive EFMM from SectionMemoryManager, it brings into EFMM extra
functionality, such as the registry of exception handling sections,
page permission management, etc. Such functionality is of no use to
llvm-bolt and can even be detrimental (see
https://github.com/llvm/llvm-project/issues/56726).

Change the base class of ExecutableFileMemoryManager to MemoryManager,
avoid registering EH sections, and skip memory finalization.

Fixes #56726


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D133994

Files:
  bolt/include/bolt/Rewrite/ExecutableFileMemoryManager.h
  bolt/lib/Rewrite/ExecutableFileMemoryManager.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D133994.460556.patch
Type: text/x-patch
Size: 5288 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220915/8d101f7e/attachment.bin>


More information about the llvm-commits mailing list