krasimir added a comment.

I also have a limited understanding based on the original patch 
https://github.com/llvm/llvm-project/commit/4db94094b469b4715d08ef37f1799bf3ea7ca8ea
 together with examining affected test cases.
I believe the intention of this section is to handle, e.g., the alignment of 
`eeeeeeeeeeeeeeeeee` and `bbbbbbbbbbbbbbbb` and `3333333333333333` in cases 
like this, where the previous toke is the `:`:

  return aaaaaaaaaaaaaaaa ? (aaaaaaaaaaaaaaaaa ? bbbbbbbbbbbbbbbbbb :
                             ccccccccccccc     ? dddddddddddddddddd :
                                                 eeeeeeeeeeeeeeeeee) :
         bbbbbbbbbbbbbbbb ? 2222222222222222 :
                            3333333333333333;

The check for `tok::question` that I remove in this context seems superfluous 
in the first place.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101033/new/

https://reviews.llvm.org/D101033

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to