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

Manuel Klimek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 29 02:38:03 PDT 2018


klimek added a comment.

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

> @klimek fair point. To be honest, I've pretty much lost interest / momentum on this feature, I very much doubt I will ever go back and re-implement from scratch as you suggest.
>  Not meaning to sound rude, I just don't want to waste anyone's time who is waiting for this (seems there are a few).


I do agree this is frustrating - I'd personally really like this feature, but having an implementation that fundamentally matches the architecture of the code is important for long-term maintenance and sustainability of the project. This is unfortunately a hard / lots-of-work feature to get right. I do appreciate the time you took to look into this.


Repository:
  rL LLVM

https://reviews.llvm.org/D28462





More information about the llvm-commits mailing list