Skip to content

[Comgr] Separate out the decision to statically link in LLVM into Comgr from the LLVM build type #2927

Description

@chinmaydd

With COMGR_STATIC_LLVM=ON, comgr sets LLVM_LINK_LLVM_DYLIB/CLANG_LINK_CLANG_DYLIB OFF near the top of the file, before find_package.

A later include(opencl_header) re-runs find_package(Clang), which re-reads the Clang/LLVM package config and flips both vars back ON — so by the time the link libraries are resolved, comgr links the dynamic libLLVM.so/libclang-cpp.so even though a static build was requested.

#2926 takes a stab at this by disabling those flipped vars again, but ideally we should separate out the Comgr choice to statically link in those libraries, from the build type of the underlying LLVM build.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions