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

Reply via email to