[PATCH] D37599: Add '\n' in ClangDataCollectorsEmitter

2017-09-08 Thread Konstantin Zhuravlyov via Phabricator via cfe-commits
kzhuravl added a comment. In https://reviews.llvm.org/D37599#864376, @arphaman wrote: > LGTM. Do you have commit access? Yep, committed. Thanks. Repository: rL LLVM https://reviews.llvm.org/D37599 ___ cfe-commits mailing list cfe-commits@lists

[PATCH] D37599: Add '\n' in ClangDataCollectorsEmitter

2017-09-08 Thread Konstantin Zhuravlyov via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL312795: Add '\n' in ClangDataCollectorsEmitter (authored by kzhuravl). Changed prior to commit: https://reviews.llvm.org/D37599?vs=114269&id=114376#toc Repository: rL LLVM https://reviews.llvm.org/D

[PATCH] D37599: Add '\n' in ClangDataCollectorsEmitter

2017-09-07 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman accepted this revision. arphaman added a comment. This revision is now accepted and ready to land. LGTM. Do you have commit access? https://reviews.llvm.org/D37599 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.or

[PATCH] D37599: Add '\n' in ClangDataCollectorsEmitter

2017-09-07 Thread Konstantin Zhuravlyov via Phabricator via cfe-commits
kzhuravl created this revision. Herald added subscribers: ilya-biryukov, wdng. Otherwise it breaks our internal windows builders: StmtDataCollectors.inc: error C2220: warning treated as error - no 'object' file generated StmtDataCollectors.inc: warning C4335: Mac file format detected: please