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

Vladislav Khmelevsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 16 12:30:29 PDT 2022


yota9 added inline comments.


================
Comment at: bolt/include/bolt/Rewrite/ExecutableFileMemoryManager.h:36
+  };
+  SmallVector<AllocInfo, 4> AllocatedSections;
+
----------------
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?


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