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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
