Higuoxing added a comment. In D85289#2196062 <https://reviews.llvm.org/D85289#2196062>, @jhenderson wrote:
> 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? Yeah, I'm ok with these names. I would like to hear more people's ideas! 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
