goldstein.w.n added a comment.

In D137181#3916505 <https://reviews.llvm.org/D137181#3916505>, @owenpan wrote:

> 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.

What did you do? Just set `PPLevel = PPBranchLevel`?


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