[flang-commits] [PATCH] D78136: [flang][docs]Doxygen support in flang

[sameeran joshi via Phabricator via flang-commits]

sameeranjoshi added a comment.

In D78136#1992084 <https://reviews.llvm.org/D78136#1992084>, @DavidTruby wrote:

> In D78136#1987218 <https://reviews.llvm.org/D78136#1987218>, @sameeranjoshi wrote:
>
> > Above doxygen patch seems to fail for me for below flags
> >
> > cmake ../llvm
> >  -DLLVM_ENABLE_DOXYGEN=ON \
> >  -DLLVM_BUILD_DOCS=OFF \
> >  -DFLANG_INCLUDE_DOCS=OFF \
> >
> > ninja install
>
>
> I'm not really sure if this combination of flags makes sense anyway, what's the expected behaviour when using these flags? I can't really tell intuitively


Expected behavior should not build nor install any documentation and documentation targets(doxygen-flang) should not be visible.

Whereas adding LLVM_INCLUDE_DOCS=OFF and MLIR_INCLUDE_DOCS=OFF seems to work fine.

cmake ../llvm
-DLLVM_ENABLE_DOXYGEN=ON \
-DLLVM_BUILD_DOCS=OFF \
-DFLANG_INCLUDE_DOCS=OFF
**-DLLVM_INCLUDE_DOCS=OFF \
-DMLIR_INCLUDE_DOCS=OFF \**

But does anyone have an idea on the appropriate flags for generating doxygen based documentation, as the number of permutations could be huge to test based on llvm and mlir flags?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78136/new/

https://reviews.llvm.org/D78136