Skip to content

[SYCL] Delete the workaround for wrong s_upsample mangling #1053

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
Feb 25, 2020
Merged

[SYCL] Delete the workaround for wrong s_upsample mangling #1053

merged 1 commit into from
Feb 25, 2020

Conversation

imashkov
Copy link
Contributor

Signed-off-by: Ilya Mashkov ilya.mashkov@intel.com

@AlexeySachkov AlexeySachkov self-assigned this Jan 24, 2020
AlexeySachkov
AlexeySachkov previously approved these changes Jan 24, 2020
Copy link
Contributor

@Fznamznon Fznamznon left a comment

Choose a reason for hiding this comment

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

Could you please rework your patch title and provide description?
What is the workaround?
What is the closed issue?
It's confusing.

@imashkov imashkov changed the title [SYCL]Deletion of workaround for closed issue [SYCL]Deletion of workaround for wrong mangling of s_upsample Jan 24, 2020
@bader bader changed the title [SYCL]Deletion of workaround for wrong mangling of s_upsample [SYCL] Delete the workaround for wrong s_upsample mangling Jan 24, 2020
@imashkov
Copy link
Contributor Author

Patch can not be merged yet, fpga emulator test fell. Close this pull request until fix is in runtime.

@imashkov imashkov closed this Jan 24, 2020
@imashkov imashkov reopened this Feb 17, 2020
@imashkov
Copy link
Contributor Author

Same pull request passed all checks in private repo, so after all successful checks here it can be merged

We had issue with wrong mangling of s_upsample. I fixed it a long time ago, so we can delete workaround now.

Signed-off-by: Ilya Mashkov <ilya.mashkov@intel.com>
@imashkov
Copy link
Contributor Author

@bader this pull request is ready to be merged, please do it, when you have time.

@AlexeySachkov
Copy link
Contributor

@bader this pull request is ready to be merged, please do it, when you have time.

It wasn't really ready without approval, fixed

@bader bader merged commit da0f66b into intel:sycl Feb 25, 2020
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.

4 participants