[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