DavidSpickett added inline comments.

================
Comment at: lldb/include/lldb/Target/RegisterFlags.h:20
+  public:
+    Field(const std::string &name, unsigned start, unsigned end,
+          const std::string &type)
----------------
DavidSpickett wrote:
> tschuett wrote:
> > There are a lot of `const std::string& ` . Are you allowed to use 
> > `llvm::StringRef` or even `std::string_view` in LLDB?
> We can use `llvm::StringRef`, but in this case this will be passed a 
> std::string so I just defaulted to the most conservative version (this 
> happens in a later patch).
> 
> I suppose you could move here, or pass by copy and assume the compiler will 
> figure it out.
> 
> 
Actually, it doesn't have to be a std::string the XML parser does give you a 
StringRef.

I will update this and the subsequent patches.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145566

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

Reply via email to