[PATCH] D45815: [libclang] Allow skipping function bodies in preamble only

Nikolai Kosjar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 27 07:18:51 PDT 2018


nik added inline comments.


================
Comment at: lib/Parse/ParseCXXInlineMethods.cpp:104
 
-  if (SkipFunctionBodies && (!FnD || Actions.canSkipFunctionBody(FnD)) &&
-      trySkippingFunctionBody()) {
+  if (SkipFunctionBodies != SkipFunctionBodiesKind::None &&
+      (!FnD || Actions.canSkipFunctionBody(FnD)) && trySkippingFunctionBody()) {
----------------
ilya-biryukov wrote:
> Can't this be a template too? Do we need to check for it here?
Correct. Here is also the point were we would check for the parent class being a template.


Repository:
  rC Clang

https://reviews.llvm.org/D45815





More information about the cfe-commits mailing list