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

Reply via email to