owenpan added a comment. In D137181#3916521 <https://reviews.llvm.org/D137181#3916521>, @goldstein.w.n wrote:
> I think we can remove the places I set `InitialPPLevel` and `OldPPLevel` but > anything else I remove causes at least one test to fail. > > What did you do? Just set `PPLevel = PPBranchLevel`? Yes, if there is a header guard. Otherwise, set `PPLevel` to `PPBranchLevel + 1`. 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