https://github.com/DavidSpickett approved this pull request.

LGTM with the braces fixed.

In theory we could wait for @emaste to comment but this seems like a clear 
usability improvement to me so I'm approving it myself. We've discussed the 
existing tests and worst case, if someone's CI fails, these patches will be 
easy to revert.

(plus we are just past the llvm release branching, so there's no danger of 
being included in a release right now)

https://github.com/llvm/llvm-project/pull/178069
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to