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