[flang-commits] [PATCH] D84334: [flang] Version information in flang/f18

[Richard Barton via Phabricator via flang-commits]

richard.barton.arm added a comment.

@coti the reason for the build failure is because you have removed both of the duplicate definitions of include in flang/tools/f18/CMakeLists.txt. This causes CMake not to be able to find the right files when it creates the builtin module files. One of these needs to be restored in your patch for the build to work as before.

I have marked the line in the original source that needs to be reinstated to your patch to make it work correctly for my in-tree build. If you re-submit a patch with that change then it can be tested out of tree.



================
Comment at: flang/tools/f18/CMakeLists.txt:32
-
-set(include ${FLANG_BINARY_DIR}/include/flang)
 
----------------
This is the line to reinstate.


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

https://reviews.llvm.org/D84334