[PATCH] D133994: [BOLT] Change base class of ExecutableFileMemoryManager
Vladislav Khmelevsky via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 16 13:06:48 PDT 2022
yota9 accepted this revision.
yota9 added inline comments.
This revision is now accepted and ready to land.
================
Comment at: bolt/include/bolt/Rewrite/ExecutableFileMemoryManager.h:36
+ };
+ SmallVector<AllocInfo, 4> AllocatedSections;
+
----------------
maksfb wrote:
> yota9 wrote:
> > It seems to me that 4 is not suitable number and might be increased significantly, maybe like 32 or even 64, what do you think?
> In relocation mode, there are not that many sections emitted. I can bump it to 8.
You're right it is too much, I was also thinking about debug sections, that might be around ~8 as I understand. But I don't insist here, thanks for bumping anyway :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133994/new/
https://reviews.llvm.org/D133994
More information about the llvm-commits
mailing list