https://github.com/JDevlieghere approved this pull request.
The change itself looks straightforward. I was going to leave a comment about the `auto` here, but this whole class' usage of `auto` is quite beyond what the LLVM style guide specifies. The golden rule is to be consistent, so don't change it here, but please considering addressing this in a separate PR (doesn't need review) to avoid setting the wrong example going forward. https://github.com/llvm/llvm-project/pull/172360 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
