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

[sameeran joshi via Phabricator via llvm-commits]

sameeranjoshi requested review of this revision.
sameeranjoshi added a reviewer: mehdi_amini.
sameeranjoshi added a comment.

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

As it tries to copy <path to build>/docs/html to <path to install prefix>/docs the same error for out-of-tree reported (here <https://github.com/flang-compiler/f18/pull/1065#issuecomment-606709553>) which I tried to fix for out-of-tree builds (here <https://github.com/flang-compiler/f18/pull/1065/commits/ea4f9612b8acac681ee64993663c964142b92304>)
Now that we are in tree, things seem to have dependency on mlir and llvm as well.

llvm-project/mlir as well shows the same error.
When adding the dependency (here <https://github.com/flang-compiler/f18/pull/1065/commits/ea4f9612b8acac681ee64993663c964142b92304>)  to llvm and mlir docs/CMakeLists.txt seems to fix error.

@sscalpone is that the expected way how the doxygen is tested in other llvm umbrella projects or am I testing it in wrong way?

@jdoerfert does clang and mlir also have the similar issue, which needs to be fixed, as adding a fix in these projects seems to solve issue in flang ?


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

https://reviews.llvm.org/D78136