dexonsmith added a comment. In D104819#2839315 <https://reviews.llvm.org/D104819#2839315>, @mstorsjo wrote:
> In D104819#2839263 <https://reviews.llvm.org/D104819#2839263>, @dexonsmith > wrote: > >> Just be sure to clang-format when you do the mechanical changes in the >> follow up patches.) > > Hmm, those would have to be manually reviewed, but I guess I can try to do > that (discarding changes where the surrounding code isn't too well formatted > now already so it does more extensive reformattings other than for the > renaming). Sounds good. (I'm generally in favour of erring on the side of trusting clang-format, especially for large refactorings like this; I wouldn't spend too much time analyzing whether it's an improvement...) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104819/new/ https://reviews.llvm.org/D104819 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
