Skip to content

Commit

Permalink
[frontend] Add missing dependencies into DAP target. (buddy-compiler#357
Browse files Browse the repository at this point in the history
)

Ninja introduced different build graph algorithm in version v1.12.0,
which will required project to explicitly specify dependencies. Without
this commit, ninja v1.12.0 will not link mlir-translate and llc before the DAP
target and causing build failure.
  • Loading branch information
Avimitin authored Aug 6, 2024
1 parent ec8a179 commit be2811c
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions frontend/Interfaces/lib/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ add_custom_command(OUTPUT DIP.o
-mattr=${BUDDY_OPT_ATTR}
--filetype=obj
-o ${CMAKE_CURRENT_BINARY_DIR}/DIP.o
DEPENDS buddy-opt
DEPENDS mlir-translate llc buddy-opt
)

add_library(BuddyLibDIP STATIC DIP.o)
Expand Down Expand Up @@ -56,7 +56,7 @@ add_custom_command(OUTPUT DAP.o
-mattr=${BUDDY_OPT_ATTR}
--filetype=obj
-o ${CMAKE_CURRENT_BINARY_DIR}/DAP.o
DEPENDS buddy-opt
DEPENDS mlir-translate llc buddy-opt
)

add_custom_command(OUTPUT DAP-extend.o
Expand All @@ -82,7 +82,7 @@ add_custom_command(OUTPUT DAP.o
-mattr=${BUDDY_OPT_ATTR}
-filetype=obj -relocation-model=pic
-o ${CMAKE_CURRENT_BINARY_DIR}/DAP-extend.o
DEPENDS buddy-opt
DEPENDS mlir-translate llc buddy-opt
)

add_library(BuddyLibDAP STATIC DAP.o DAP-extend.o)
Expand Down Expand Up @@ -115,7 +115,7 @@ SET_TARGET_PROPERTIES(BuddyLibDAP PROPERTIES
-mattr=${BUDDY_OPT_ATTR}
-filetype=obj
-o ${CMAKE_CURRENT_BINARY_DIR}/DAPVectorization.o
DEPENDS buddy-opt
DEPENDS mlir-translate llc buddy-opt
)

add_library(BuddyLibDAPVectorization STATIC DAPVectorization.o)
Expand Down

0 comments on commit be2811c

Please sign in to comment.