[PATCH] D141234: [BOLT][Golang support] Core changes

[Thorsten via Phabricator via llvm-commits]

tschuett added a comment.

https://llvm.org/docs/ProgrammersManual.html#picking-the-right-data-structure-for-a-task



================
Comment at: bolt/include/bolt/Core/BinaryContext.h:237
 public:
+  std::vector<std::string> ExtraSectionsNames;
+
----------------
Do you care about the order? Or is a SmallStringSet sufficient?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141234