intel / intel/llvm

SYCL introduces unnecessary(?) dependency on Sema

Open
#3,309 5 comments 0 reactions 0 assignees View on GitHub
confirmed
Dominant language
LLVM
Stars
1.5k
Forks
854
Avg merge
3d 17h
Merged PRs (30d)
137

Description

https://github.com/intel/llvm/pull/3301 added Sema library as a dependency for Frontend, CodeGen and a couple of other libraries, which doesn't make sense to me. According to @DenisBakhvalov, it required to build clang in "shared libraries" configuration.

This looks very suspicious. Community version doesn't have this dependency. https://github.com/llvm/llvm-project/blob/main/clang/lib/CodeGen/CMakeLists.txt#L92
+@erichkeane, @AaronBallman, do you know the reason for clang's CodeGen to depend on Sema library?

_Originally posted by @bader in https://github.com/intel/llvm/pull/3301#discussion_r588063845_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.