[tests][modernize]Make the tests compile with hipSYCL#4
[tests][modernize]Make the tests compile with hipSYCL#4sbalint98 wants to merge 5 commits intoAdaptiveCpp:masterfrom
Conversation
*Changed e.get_cl_code to e.what *Fixed namespace sycl:: to cl::sycl *Gave explict type to exception_handler
| const int *n, const int *k, const fp *alpha, const fp *a, const int *lda, | ||
| const fp *b, const int *ldb, const fp *beta, fp *c, const int *ldc); | ||
|
|
||
| #ifdef NOT_HIPSYCL |
There was a problem hiding this comment.
could you use the ENABLE_HALF_ROUTINES here and get rid of the explicit NOT_HIPSYCL macro?
or instead just use a macro that is defined by hipSYCL anyways?
There was a problem hiding this comment.
Ohh.. this shouldn't be here at all... this should be in #6 ... Thanks for noticing :) But I think I'll let this be here, instead of moving it over to the other PR
|
|
||
| #define POISSON_ARGS 0.5 | ||
|
|
||
| using namespace cl; |
There was a problem hiding this comment.
Why do you need to open the cl namespace if you then qualify everything with cl:: in the code below?
There was a problem hiding this comment.
I don't see any good reason to do it so. I will remove the qualifications from below, to reduce the diff.
| int test(device *dev, oneapi::mkl::layout layout, int64_t group_count) { | ||
| // Catch asynchronous exceptions. | ||
| auto exception_handler = [](exception_list exceptions) { | ||
| cl::sycl::async_handler exception_handler = [](exception_list exceptions) { |
There was a problem hiding this comment.
Why is it even necessary to not use auto? I think it should also work with auto.
There was a problem hiding this comment.
Mhmm, it doesn't seem to be necessary anymore... I recall that a few weeks ago there was an issue with hipSYCL regarding this.
3 participants
This PR concenrs the tests. The following two changes were made:
cl::sycl::async_handlerinstead ofautotype.what()instead of usingget_cl_code()