[Lldb-commits] [lldb] [llvm] [Obj2Yaml] Add support for minidump generation with 64b memory ranges. (PR #101272)

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Mon Aug 5 02:05:40 PDT 2024


================
@@ -132,6 +140,71 @@ class MinidumpFile : public Binary {
     size_t Stride;
   };
 
+  class Memory64ListFacade {
----------------
labath wrote:

I don't think you need this facade class (particularly if we can avoid storing the of the vector<MemoryDescriptor_64> thing). llvm APIs of this kind will typically return `iterator_range<YourIterator>` and put all of the logic into the iterator class.

https://github.com/llvm/llvm-project/pull/101272


More information about the lldb-commits mailing list