[PATCH] D41102: Setup clang-doc frontend framework

[Roman Lebedev via Phabricator via cfe-commits]

lebedev.ri added inline comments.


================
Comment at: clang-doc/Representation.h:117
+  bool IsDefinition = false;
+  llvm::Optional<Location> DefLoc;
+  llvm::SmallVector<Location, 2> Loc;
----------------
I meant that `IsDefinition` controls whether `DefLoc` will be set/used or not.
So with `llvm::Optional<Location> DefLoc`, you don't need the `bool IsDefinition`.


https://reviews.llvm.org/D41102