[PATCH] D43341: [clang-doc] Implement reducer portion of the frontend framework

[Julie Hockett via Phabricator via cfe-commits]

juliehockett added inline comments.


================
Comment at: clang-doc/tool/ClangDocMain.cpp:181
+        doc::writeInfo(I.get(), Buffer);
+      if (DumpResultToFile("bc", Group.getKey() + ".bc", Buffer))
+        return 1;
----------------
ioeric wrote:
> juliehockett wrote:
> > ioeric wrote:
> > > (Sorry that I might be missing context here.)
> > > 
> > > Could you please explain the incentive for dumping each info group to one bc file? If the key identifies a symbol (e.g. USR), wouldn't this result in a bitcode file being created for each symbol? This doesn't seem very scalable.  
> > Yes, it would. This is mostly for debugging, since there's not really any tools outside the clang system that would actually want/be able to use this information.
> Ok, is there any plan to convert intermediate result to final result and output in a more accessible format? If so, please add a `FIXME` somewhere just to be clearer.
That's what the next patch set is (to generate YAML).


https://reviews.llvm.org/D43341