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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to