[PATCH] D50628: [Preamble] Empty preamble is not an error.

2018-08-17 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC340029: [Preamble] Empty preamble is not an error. (authored by hokein, committed by ). Changed prior to commit: https://reviews.llvm.org/D50628?vs=160319&id=161241#toc Repository: rC Clang https://

[PATCH] D50628: [Preamble] Empty preamble is not an error.

2018-08-13 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land. LGTM Repository: rC Clang https://reviews.llvm.org/D50628 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.l

[PATCH] D50628: [Preamble] Empty preamble is not an error.

2018-08-13 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: ilya-biryukov. Empty preamble is valid for source file which doesn't have any preprocessor and #includes. This patch makes clang treat an empty preamble as a normal preamble. Check: ninja check-clang A testcase is added in https://reviews.l