Skip to content

Fix ClangOpenCLBuiltinsImpl target not found #15

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 1 commit into from

Conversation

duzy
Copy link

@duzy duzy commented Jun 14, 2019

set_source_files_properties OBJECT_DEPENDS for SemaLookup.cpp does not work as it's actually looking for a file target rather then a custom (clang_tablegen) target. The add_dependencies is going to fix the dependency error.

See also llvm-mirror/clang#63 https://github.com/llvm-mirror/clang/pull/63/files

set_source_files_properties OBJECT_DEPENDS for SemaLookup.cpp does not work as it's actually looking for a file target rather then a custom (clang_tablegen) target. The add_dependencies is going to fix the dependency error.

See also llvm-mirror/clang#63 https://github.com/llvm-mirror/clang/pull/63/files
@kocsenc
Copy link

kocsenc commented Jun 19, 2019

From the GitHub landing page for the repository:

This is the canonical git mirror of the LLVM subversion repository. The repository does not accept github pull requests at this moment. Please submit your patches at http://reviews.llvm.org. http://llvm.org

Please check out the How to submit a patch section of the docs.

@asl asl closed this Jun 19, 2019
@repo-lockdown repo-lockdown bot locked and limited conversation to collaborators Jun 27, 2019
cmtice referenced this pull request May 3, 2023
https://llvm.org/docs/LangRef.html#llvm-powi-intrinsic
The max length of the integer power of `llvm.powi` intrinsic is 32, and
the value can be negative. If we use `int32_t` to store this value, `-Val`
will underflow when it is `INT32_MIN`

The issue was reported in D149033.
jpinot pushed a commit to jpinot/llvm-project that referenced this pull request Apr 9, 2025
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants