DavidSpickett added a comment. @omjavaid Any more comments on this?
================ Comment at: lldb/include/lldb/Core/Architecture.h:110 + virtual const MemoryTagManager *GetMemoryTagManager() const { + return nullptr; + } ---------------- I also had the thought that we could just return instances of MemoryTagManager, however we rely on virtual functions for the implementation. (incredibly obvious once you try it, but would have been nice to remove a bunch of pointer indirection) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95602/new/ https://reviews.llvm.org/D95602 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits