Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Mangling of pipe is broken for Windows #41905

Open
llvmbot opened this issue Jul 10, 2019 · 2 comments
Open

Mangling of pipe is broken for Windows #41905

llvmbot opened this issue Jul 10, 2019 · 2 comments
Labels
bugzilla Issues migrated from bugzilla OpenCL

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Jul 10, 2019

Bugzilla Link 42560
Version unspecified
OS other
Reporter LLVM Bugzilla Contributor
CC @AnastasiaStulova,@rnk

Extended Description

As mentioned in http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20190708/278270.html, the pipe types are not properly mangled on Windows.

C:\b\slave\clang-x64-windows-msvc\build\llvm.src\tools\clang\test\CodeGenOpenCL\pipe_builtin.cl:9:1:
error: cannot mangle this OpenCL pipe type yet

void test1(read_only pipe int p, global int *ptr) {

^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

1 error generated.

It's unlikely related to D64074, which added the test that triggered the bug elsewhere: unlike in C mode, in C++ mode we are mangling user-defined functions.
@llvmbot
Copy link
Collaborator Author

llvmbot commented Jul 11, 2019

See also https://reviews.llvm.org/rC365557

@AnastasiaStulova
Copy link
Contributor

FYI this is also an issue for OpenCL C with overloadable attribute used to declare BIFs.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bugzilla Issues migrated from bugzilla OpenCL
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@AnastasiaStulova @llvmbot