[PATCH] D129519: [Symbolizer] Implement contextual symbolizer markup elements.

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 11 15:09:06 PDT 2022


phosek added inline comments.


================
Comment at: llvm/include/llvm/DebugInfo/Symbolize/MarkupFilter.h:128-129
+
+  std::map<uint64_t, Module> Modules;
+  std::map<uint64_t, MMap> MMaps;
 };
----------------
I'd recommend using `DenseMap` which is more performant than `std::map`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129519



More information about the llvm-commits mailing list