Skip to content

Commit

Permalink
Fixes a problem when LLVM_DEFINITIONS contains a definition with an `…
Browse files Browse the repository at this point in the history
…=` in it.

add_definitions doesn't seem to parse this type of argument correctly.
  • Loading branch information
vmware authored and yonghong-song committed Aug 12, 2020
1 parent 9ef20e7 commit 3fbb664
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/cc/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,11 @@ include_directories(${LIBELF_INCLUDE_DIRS})
# todo: if check for kernel version
include_directories(${CMAKE_CURRENT_SOURCE_DIR}/libbpf/include)
include_directories(${CMAKE_CURRENT_SOURCE_DIR}/libbpf/include/uapi)
add_definitions(${LLVM_DEFINITIONS})

# add_definitions has a problem parsing "-D_GLIBCXX_USE_CXX11_ABI=0", this is safer
separate_arguments(LLVM_DEFINITIONS)
add_compile_options(${LLVM_DEFINITIONS})

configure_file(libbcc.pc.in libbcc.pc @ONLY)
configure_file(bcc_version.h.in bcc_version.h @ONLY)

Expand Down

0 comments on commit 3fbb664

Please sign in to comment.