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

[SYCL-MLIR] Add parallel_for.cpp #1374

Merged
merged 3 commits into from
Nov 10, 2022
Merged

[SYCL-MLIR] Add parallel_for.cpp #1374

merged 3 commits into from
Nov 10, 2022

Conversation

whitneywhtsang
Copy link

Signed-off-by: Tsang, Whitney [email protected]

Signed-off-by: Tsang, Whitney <[email protected]>
@whitneywhtsang whitneywhtsang added the sycl-mlir Pull requests or issues for sycl-mlir branch label Nov 10, 2022
@whitneywhtsang whitneywhtsang self-assigned this Nov 10, 2022
Copy link

@victor-eds victor-eds left a comment

Choose a reason for hiding this comment

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

LGTM but nit

Signed-off-by: Tsang, Whitney <[email protected]>
Signed-off-by: Tsang, Whitney <[email protected]>
@whitneywhtsang whitneywhtsang merged commit 3800ffd into sycl-mlir Nov 10, 2022
@whitneywhtsang whitneywhtsang deleted the parallel_for branch November 10, 2022 14:04
@etiotto
Copy link

etiotto commented Nov 10, 2022

For the record IMO we should not add test cases that fail because it makes PRs appear as "red". The correct process is to add test coverage only after the PR that makes the test case pass has been merged in.

whitneywhtsang added a commit to intel/llvm that referenced this pull request Nov 10, 2022
Globals of `ext_vector_type` type used to represent as `memref<3xi64>`,
which lowers to array type pointer in LLVM.
This can cause link failures when files are compiled from different
compilers.

This PR changes globals of `ext_vector_type` type to be represented as
`memref<vector<3xi64>>`, which lowers to vector type pointer, the same
type as directly compile from `clang`.

Note: After this PR, `parallel_for.cpp` can run successfully. Test case
added in intel/llvm-test-suite#1374.

Signed-off-by: Tsang, Whitney <[email protected]>
whitneywhtsang added a commit that referenced this pull request Jan 13, 2023
* [SYCL-MLIR] Add parallel_for.cpp

Signed-off-by: Tsang, Whitney <[email protected]>
whitneywhtsang added a commit that referenced this pull request Jan 18, 2023
* [SYCL-MLIR] Add parallel_for.cpp

Signed-off-by: Tsang, Whitney <[email protected]>
whitneywhtsang added a commit that referenced this pull request Jan 25, 2023
* [SYCL-MLIR] Add parallel_for.cpp

Signed-off-by: Tsang, Whitney <[email protected]>
whitneywhtsang added a commit that referenced this pull request Jan 27, 2023
* [SYCL-MLIR] Add parallel_for.cpp

Signed-off-by: Tsang, Whitney <[email protected]>
whitneywhtsang added a commit that referenced this pull request Feb 17, 2023
* [SYCL-MLIR] Add parallel_for.cpp

Signed-off-by: Tsang, Whitney <[email protected]>
whitneywhtsang added a commit that referenced this pull request Mar 1, 2023
* [SYCL-MLIR] Add parallel_for.cpp

Signed-off-by: Tsang, Whitney <[email protected]>
whitneywhtsang added a commit that referenced this pull request Mar 27, 2023
* [SYCL-MLIR] Add parallel_for.cpp

Signed-off-by: Tsang, Whitney <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
sycl-mlir Pull requests or issues for sycl-mlir branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants