Skip to content

[NFC][SYCL] Add todo comment for validation of group-local variables definintion #3437

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

Conversation

mlychkov
Copy link
Contributor

Signed-off-by: Mikhail Lychkov mikhail.lychkov@intel.com

@mlychkov mlychkov requested review from bader and kbobrovs March 29, 2021 07:35
@mlychkov mlychkov force-pushed the private/mlychkov/private/mlychkov/kernel_local_mem_pass_todo branch from 654a8c7 to f94e8b6 Compare March 30, 2021 10:12
@mlychkov
Copy link
Contributor Author

@kbobrovs Could you please review this patch?

Failed test is not related to my changes and is a known issue.

@bader
Copy link
Contributor

bader commented Apr 13, 2021

@kbobrovs Could you please review this patch?

Failed test is not related to my changes and is a known issue.

@mlychkov, please, disable failing tests and re-run the job to get clean results.

@mlychkov
Copy link
Contributor Author

@kbobrovs Could you please review this patch?
Failed test is not related to my changes and is a known issue.

@mlychkov, please, disable failing tests and re-run the job to get clean results.

Looks like failed test was already disabled: intel/llvm-test-suite#205

…definintion

Signed-off-by: Mikhail Lychkov <mikhail.lychkov@intel.com>
@mlychkov mlychkov force-pushed the private/mlychkov/private/mlychkov/kernel_local_mem_pass_todo branch from f94e8b6 to fcbe5e6 Compare April 14, 2021 07:20
@mlychkov
Copy link
Contributor Author

@kbobrovs friendly ping

@bader bader merged commit 8aae3d3 into intel:sycl Apr 20, 2021
@mlychkov mlychkov deleted the private/mlychkov/private/mlychkov/kernel_local_mem_pass_todo branch April 21, 2021 12:45
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