sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.

Can't say I really understand this (either the before/after state).

It seems like we're trying to vertically-align a bunch of operands to a ?: 
chain, and this patch stops aligning the ones that follow a ? in certain cases. 
But I don't understand why this is correct, and what the point is of aligning 
(only) the ones following `:`.

Is there an intuitive explanation?


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