[PATCH] D79388: [clang-format] Fix AlignConsecutive on PP blocks
Jake Merdich via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 21 08:42:32 PDT 2020
JakeMerdichAMD added a comment.
I'd be very surprised if any of the tests included in this change pass with that line commented.... it's meant so that things like #if and #else properly separate alignment after the first preprocessor run, where the whitespace manager doesn't have the full context of things between it.
This will break with any PP statement though, not just if-elif-else-endif. I'll try moving it to somewhere more specific like the cases in parsePPDirective and put out a fix.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79388/new/
https://reviews.llvm.org/D79388
More information about the cfe-commits
mailing list