squiid/llvm
1
0
Fork 0
Code Issues Pull requests Activity

Remove linking all targets from mlir-opt (NFC)

Browse source 
There is no need to directly depends on this from mlir-opt, some library
may transitively depend on a subset of the targets when enabled (like
NVPTX for Cuda codegen tests) but this is handled by CMake already.
This commit is contained in:
Mehdi Amini 2020-04-01 17:19:26 +00:00
parent 0dd21130ef
commit b8c260c38d
1 changed files with 0 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
mlir/tools/mlir-opt/CMakeLists.txt
View file

@ -47,16 +47,6 @@ target_link_libraries(MLIRMlirOptMain
add_llvm_tool(mlir-opt
mlir-opt.cpp
)
# Manually expand the target library, since our MLIR libraries
# aren't plugged into the LLVM dependency tracking. If we don't
# do this then we can't insert the CodeGen library after ourselves
llvm_expand_pseudo_components(TARGET_LIBS AllTargetsCodeGens)
# Prepend LLVM in front of every target, this is how the library
# are named with CMake
SET(targets_to_link)
FOREACH(t ${TARGET_LIBS})
LIST(APPEND targets_to_link "LLVM${t}")
ENDFOREACH(t)
llvm_update_compile_flags(mlir-opt)
target_link_libraries(mlir-opt PRIVATE ${LIBS} ${targets_to_link})