labath added inline comments.

================
Comment at: lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp:4619
+    for (uint32_t &x : remote_reg_info.value_regs) {
+      std::map<uint32_t, uint32_t>::iterator x_lldb =
+          remote_to_local_map.find(x);
----------------
auto :)


================
Comment at: 
lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp:4626-4629
+      std::map<uint32_t, uint32_t>::iterator x_lldb =
+          remote_to_local_map.find(x);
+      x = x_lldb != remote_to_local_map.end() ? x_lldb->second
+                                              : LLDB_INVALID_REGNUM;
----------------
Make this a lambda and then do 
`llvm::transform(remote_reg_info.invalidate_regs, 
remote_reg_info.invalidate_regs.begin(), lambda)`


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

https://reviews.llvm.org/D110027

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to