[PATCH] Added doxygen support for the tools of clang-tools-extra repo

Sean Silva silvas at purdue.edu
Fri Feb 22 09:55:53 PST 2013


  Ugh. This is just repeating the same mistakes that are currently present in the llvm and clang doxygen setups.

  In particular, the Makefile is unusable for CMake builds (and AFAIK nobody uses the functionality in that Makefile anyway). It also uses LLVM's weird doxygen stylesheet instead of the default one which is *much* easier to navigate.

  We should really just have a single Doxyfile (maybe Doxyfile.in, but make it work with CMake and Make). I see no reason to add anything else. You can look at <https://github.com/chisophugis/klang/tree/master/docs> for an example of my current preference for the layout of the `docs/` directory.

  I also noticed that the Sphinx Makefile is called Makefile.sphinx. I've added some post-commit review to the commit that added that (I should have noticed this earlier during pre-commit review).

  Let's avoid making the mistakes of the past.

http://llvm-reviews.chandlerc.com/D451



More information about the cfe-commits mailing list