jhenderson added a subscriber: labath.
jhenderson added a comment.

I see the point, but we don't do it for all fields in other contexts, and I 
have some mild concerns that `DebugAbbrevOffset` is unnecessarily verbose (I'd 
think `AbbrevOffset` would be sufficient. Perhaps it would be best to draw in 
one or two others? @JDevlieghere / @labath, any thoughts?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85289

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

Reply via email to