Skip to content

Conversation

elizabethandrews
Copy link
Contributor

Add support for KernelNameType with is a template specialization class with template template arguments which in turn is templated using enum.

Signed-off-by: Elizabeth Andrews [email protected]

with template template arguments which in turn is templated using enum.

Signed-off-by: Elizabeth Andrews <[email protected]>
@erichkeane
Copy link
Contributor

@premanandrao please take a look for me.

premanandrao
premanandrao previously approved these changes Jun 2, 2020
Copy link
Contributor

@premanandrao premanandrao left a comment

Choose a reason for hiding this comment

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

LGTM

erichkeane
erichkeane previously approved these changes Jun 2, 2020
@elizabethandrews elizabethandrews dismissed stale reviews from erichkeane and premanandrao via 75e846f June 2, 2020 19:04
@elizabethandrews
Copy link
Contributor Author

@bader I don't think this Jenkins issue is related to my patch.

@bader
Copy link
Contributor

bader commented Jun 3, 2020

@bader I don't think this Jenkins issue is related to my patch.

Okay. Do you know who is working on addressing these issues?

@elizabethandrews
Copy link
Contributor Author

@bader I don't think this Jenkins issue is related to my patch.

Okay. Do you know who is working on addressing these issues?

No I do not. I see it is failing for other PRs too. I'm not sure how to investigate it.

@bader bader merged commit 125b05c into intel:sycl Jun 3, 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.

5 participants