[PATCH] D41189: [Frontend] Treat function with skipped body as definition

2017-12-14 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC320696: [Frontend] Treat function with skipped body as definition (authored by ibiryukov, committed by ). Changed prior to commit: https://reviews.llvm.org/D41189?vs=126787&id=126932#toc Repository:

[PATCH] D41189: [Frontend] Treat function with skipped body as definition

2017-12-13 Thread Serge Pavlov via Phabricator via cfe-commits
sepavloff accepted this revision. sepavloff added a comment. This revision is now accepted and ready to land. LGTM. Thanks! Repository: rC Clang https://reviews.llvm.org/D41189 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists

[PATCH] D41189: [Frontend] Treat function with skipped body as definition

2017-12-13 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov updated this revision to Diff 126787. ilya-biryukov added a comment. - Reverted accidental whitespace change. Repository: rC Clang https://reviews.llvm.org/D41189 Files: include/clang/AST/Decl.h test/Index/skipped_function_bodies.cpp Index: test/Index/skipped_function_bod

[PATCH] D41189: [Frontend] Treat function with skipped body as definition

2017-12-13 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov created this revision. ilya-biryukov added reviewers: bkramer, sepavloff. This fixes an invalid warning about missing definition of a function when parsing with SkipFunctionBodies=true Repository: rC Clang https://reviews.llvm.org/D41189 Files: include/clang/AST/Decl.h test