Skip to content

Revert "In MSVC compatibility mode, friend function declarations beha… #6143

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

Merged
merged 1 commit into from
May 12, 2022

Conversation

pvchupin
Copy link
Contributor

…ve as function declarations"

This reverts commit ad47114.

Temporary workaround for #6142

Reverting while there is a discussion at https://reviews.llvm.org/D124613
Will return back with the appropriate fix from community as soon as done.

…ve as function declarations"

This reverts commit ad47114.

Temporary workaround for intel#6142

Reverting while there is a discussion at https://reviews.llvm.org/D124613
Will return back with the appropriate fix.
@elizabethandrews
Copy link
Contributor

Is the revert required along with #6144?

@pvchupin
Copy link
Contributor Author

@elizabethandrews, #6144 is another possible way to fix the problem. Actually #6144 would be cleaner, since don't need to do temporary revert, but there are more related problems it seem. Posted there.

I think we should proceed with this revert for now.

@pvchupin pvchupin merged commit 858d04d into intel:sycl May 12, 2022
@pvchupin pvchupin deleted the fix-post-commit-win branch May 12, 2022 16:19
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