Skip to content

Commit 6081a3f

Browse files
jsjiKornevNikita
authored andcommitted
Fix libclc build
PR #1091 has refactor the logic of prepare_builtins into utils/CMakeLists.txt. We don't need to duplicate logic in CMakeLists.txt now. Before we upstream the utils/CMakeLists.txt, we should just remove the duplicate code.
1 parent 2b203c7 commit 6081a3f

File tree

1 file changed

+0
-17
lines changed

1 file changed

+0
-17
lines changed

libclc/CMakeLists.txt

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -168,23 +168,6 @@ enable_language( CLC LLAsm )
168168
# Configure prepare_builtins
169169
add_subdirectory(utils)
170170

171-
# This needs to be set before any target that needs it
172-
# We need to use LLVM_INCLUDE_DIRS here, because if we are linking to an
173-
# llvm build directory, this includes $src/llvm/include which is where all the
174-
# headers are not $build/include/ which is what LLVM_INCLUDE_DIR is set to.
175-
include_directories( ${LLVM_INCLUDE_DIRS} )
176-
177-
# Setup prepare_builtins tools
178-
set(LLVM_LINK_COMPONENTS
179-
BitReader
180-
BitWriter
181-
Core
182-
)
183-
add_llvm_executable( prepare_builtins utils/prepare-builtins.cpp )
184-
target_compile_definitions( prepare_builtins PRIVATE ${LLVM_VERSION_DEFINE} )
185-
# These were not properly reported in early LLVM and we don't need them
186-
target_compile_options( prepare_builtins PRIVATE -fno-rtti -fno-exceptions )
187-
188171
# Setup arch devices
189172
set( r600--_devices cedar cypress barts cayman )
190173
set( amdgcn--_devices tahiti )

0 commit comments

Comments
 (0)