[Lldb-commits] [PATCH] D122859: [trace] Show ideas for the main interfaces for new HTR

David Carrillo Cisneros via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Apr 1 08:03:02 PDT 2022

davidca added inline comments.

Comment at: lldb/include/lldb/Target/TraceHTR.h:73
+  //   trace
+  struct HTRInMemoryBlock {
Just like in the original HTR design, if you remove this abstraction, and store indices as indices in each layer, then you don't have to create separate versions for in memory and in disk, as the layers become very easily serializable.

Comment at: lldb/include/lldb/Target/TraceHTR.h:118
+  // growing
+  std::vector<HTRBlockStorage> m_blocks;
This does not deduplicate blocks. If the same block appears multiple times (as it's common in loops), it will be stored multiple times, consuming more memory and making pattern detection and other post-processing more time consuming.

Note that this is the original motivation of the separation between BlockDefs and bids (the trace of block ids) in the original HTR doc.

  rG LLVM Github Monorepo



More information about the lldb-commits mailing list