DavidSpickett added inline comments.
================
Comment at: lldb/source/Target/Process.cpp:6024
+llvm::Expected<const MemoryTagManager *>
+Process::GetMemoryTagManager(lldb::addr_t addr, lldb::addr_t end_addr) {
+ Architecture *arch = GetTarget().GetArchitecturePlugin();
----------------
While writing "memory tag write" I realised it saves some effort if you return
the manager and the checked range here. However it's way more useful for tag
write so I'll leave this as is for now unless you really want to see it in this
change.
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
