owenpan added a comment. In D137181#3916488 <https://reviews.llvm.org/D137181#3916488>, @goldstein.w.n wrote:
> In D137181#3916474 <https://reviews.llvm.org/D137181#3916474>, @owenpan wrote: > >> You can still simply the changes to UnwrappedLineParser.cpp a lot. In fact, >> instead of adjusting `PPLevel` at various places, you only need to set it >> once when `InMacroBody` is set. > > I don't think thats right, it needs to stay updated with the true macro depth > (even though we only use `PPLevel` for formatting `InMacroBody` lines). My experiment from a few days ago was successful and also passes all the new tests you just added here. If your experience told you otherwise, then perhaps there are some missing test cases. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137181/new/ https://reviews.llvm.org/D137181 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
