clayborg added inline comments.

================
Comment at: source/Plugins/ObjectFile/Breakpad/BreakpadRecords.cpp:393
+  llvm::StringRef LHS, RHS;
+  while (std::tie(Str, Line) = getToken(Line), !Str.empty()) {
+    if (Str.back() == ':') { // regN
----------------
clayborg wrote:
> Do we really need to pull the content apart into separate strings for each 
> register? Seems like a lot of work and 99% of these we will never accessed. 
> Maybe just store the entire string for all registers and be done? 
You can add an iterator method to the StackCFIRecord record maybe for when you 
do want to parse each register?


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

https://reviews.llvm.org/D60268



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

Reply via email to