[PATCH] D78136: [flang][docs]Doxygen support in flang
David Truby via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 22 07:01:00 PDT 2020
DavidTruby added a comment.
In D78136#1992514 <https://reviews.llvm.org/D78136#1992514>, @sameeranjoshi wrote:
> 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.
Are you sure? I would have thought LLVM_ENABLE_DOXYGEN does expose the doxygen-flang target. But again I don't really have much of an idea.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78136/new/
https://reviews.llvm.org/D78136
More information about the llvm-commits
mailing list