[PATCH] D28462: clang-format: Add new style option AlignConsecutiveMacros

Timothée Fivaz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 11 23:20:24 PDT 2018


TimF added a comment.

In https://reviews.llvm.org/D28462#1064789, @enyquist wrote:

> As far as I know, there are no updates required from me for this pull request-- I rebased on the main trunk recently, and will do it again tonight to be sure. So it should be compiling/working just fine.
>  I believe it is just awaiting final approval from somebody.


I have done a checkout of both trunks yesterday, applied your patch and compiled everything. It is working flawlessly and now the defines are correctly aligned. I still hope they'll merge it because I prefer using an official release, but until then it will do the job. 
Thanks a lot for your work on this feature, it was the last point preventing us from using clang-format. (embedded development too, so we have lots of defines). And thanks to all the LLVM contributors by the way :-)


Repository:
  rL LLVM

https://reviews.llvm.org/D28462





More information about the cfe-commits mailing list