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 :-)
cfe-commits mailing list