Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL] Tests for vec/marray math #1002

Merged
merged 25 commits into from
Dec 2, 2022
Merged

Conversation

JackAKirk
Copy link

@JackAKirk JackAKirk commented Apr 21, 2022

Tests for marray/vec SYCL math functions from: intel/llvm#6038

Signed-off-by: jack.kirk jack.kirk@codeplay.com

@JackAKirk JackAKirk changed the title tests for vec/marray math [SYCL] tests for vec/marray math Apr 26, 2022
@JackAKirk JackAKirk changed the title [SYCL] tests for vec/marray math [SYCL] Tests for vec/marray math Apr 26, 2022
@JackAKirk JackAKirk marked this pull request as ready for review April 26, 2022 10:56
@JackAKirk JackAKirk requested a review from a team as a code owner April 26, 2022 10:56
@JackAKirk
Copy link
Author

This PR includes a small patch adding marray coverage to the following not yet merged PR #895

@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

Copy link

@steffenlarsen steffenlarsen left a comment

Choose a reason for hiding this comment

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

Just some minor comments. The tests look good.

SYCL/DeviceLib/built-ins/ext_native_math.cpp Outdated Show resolved Hide resolved
SYCL/DeviceLib/half_precision_math_test_marray_vec.cpp Outdated Show resolved Hide resolved
sycl::math, native and half_precision cases covered.

Signed-off-by: jack.kirk <jack.kirk@codeplay.com>
Signed-off-by: jack.kirk <jack.kirk@codeplay.com>
Signed-off-by: jack.kirk <jack.kirk@codeplay.com>
@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

SYCL/Basic/half_builtins.cpp Outdated Show resolved Hide resolved
SYCL/DeviceLib/math_test_marray_vec.cpp Outdated Show resolved Hide resolved
Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
steffenlarsen
steffenlarsen previously approved these changes Jun 16, 2022
Copy link

@steffenlarsen steffenlarsen left a comment

Choose a reason for hiding this comment

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

LGTM!

Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
@steffenlarsen
Copy link

/verify with intel/llvm#6038

steffenlarsen
steffenlarsen previously approved these changes Jun 16, 2022
Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

4 similar comments
@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

1 similar comment
@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

@JackAKirk
Copy link
Author

/verify with intel/llvm#6038

Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
bader pushed a commit to intel/llvm that referenced this pull request Dec 2, 2022
This PR aims to fix issue: #5991
and provide efficient working marray math function implementations for
all backends.

marray math function support is currently switched on for {n} ({n}
defined in #5991) but the implementations are currently broken and
untested. There is also very limited test coverage for sycl::vec cases.
The SYCL 2020 specification states that the set {N} ({N} defined in
#5991) should be supported for marray math function cases.

All SYCL 2020 math, native math, and half_precision math functions now
have marray support when the function's arguments are of type `genfloat`
and have the same argument type for all arguments.

Tests: intel/llvm-test-suite#1002

Signed-off-by: jack.kirk <jack.kirk@codeplay.com>
Co-authored-by: JackAKirk <chezjakirk@gmail.com>
Co-authored-by: Jakub Chlanda <jakub@codeplay.com>
@bader bader merged commit c873dbc into intel:intel Dec 2, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
Tests for marray/vec SYCL math functions from: intel/llvm#6038

Signed-off-by: jack.kirk <jack.kirk@codeplay.com>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Tests for marray/vec SYCL math functions from: intel#6038

Signed-off-by: jack.kirk <jack.kirk@codeplay.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants