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.

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

Fixed build break on Debian 11 with recent OS SYCL bundle #1118

Merged
merged 1 commit into from
Mar 13, 2023

Conversation

oleksandr-pavlyk
Copy link
Collaborator

@oleksandr-pavlyk oleksandr-pavlyk commented Mar 13, 2023

Renamed inserter to __appender, moved functions into concat_impl namespace

For open-source SYCL bundle based on clang 17.0 and using GCC 10.2 toolchain compilation error about ambiguous call to inserter was encountered. This was because of name clash between the definition in this file and std::inserter.

Renaming helped resolve that.

Thank you @ZzEeKkAa for reporting this issue.

  • Have you provided a meaningful PR description?
  • Have you added a test, reproducer or referred to an issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • If this PR is a work in progress, are you filing the PR as a draft?

…space

For open-source SYCL bundle based on clang 17.0 and using GCC 10.2 toolchain
compilation error about ambiguous call to inserter was encountered. This was
because of name clash between the definition in this file and std::inserter.

Renaming helped resolve that.

Thank you @ZzEeKkAa for reportign this issue.
Copy link
Contributor

@ZzEeKkAa ZzEeKkAa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you for fixing it. I'm not sure if the issue is directly related to GCC headers. Issue was presented on Debian bullseye (GCC 10.2) and Debian bust (GCC 8.) but was not presented on Ubuntu 20.04 (GCC 9.) and Ubuntu 22.04 (GCC 11.*).

@github-actions
Copy link

@github-actions
Copy link

Array API standard conformance tests for dpctl=0.14.2=py310h76be34b_30 ran successfully.
Passed: 46
Failed: 788
Skipped: 280

@coveralls
Copy link
Collaborator

Coverage Status

Coverage: 82.404%. Remained the same when pulling b42c511 on fix-build-break into 51863d4 on master.

@oleksandr-pavlyk oleksandr-pavlyk merged commit 8f828f2 into master Mar 13, 2023
@oleksandr-pavlyk oleksandr-pavlyk deleted the fix-build-break branch March 13, 2023 23:10
@github-actions
Copy link

Deleted rendered PR docs from intelpython.github.com/dpctl, latest should be updated shortly. 🤞

@github-actions
Copy link

Array API standard conformance tests for dpctl=0.14.2=py310h76be34b_39 ran successfully.
Passed: 46
Failed: 788
Skipped: 280

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants