Skip to content
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

[GPU] Register initial GPU pipeline that uses IMEX #329

Merged
merged 10 commits into from
Sep 16, 2024

Conversation

dchigarev
Copy link
Contributor

@dchigarev dchigarev commented Sep 9, 2024

Register the current linalg-xegpu-gpu_bin pipeline as is.

Signed-off-by: dchigarev <dmitry.chigarev@intel.com>
Signed-off-by: dchigarev <dmitry.chigarev@intel.com>
Signed-off-by: dchigarev <dmitry.chigarev@intel.com>
@dchigarev dchigarev added the GPU label Sep 9, 2024
@lmontigny lmontigny added this to the 0.2 GPU Downstream IMEX milestone Sep 9, 2024
Signed-off-by: dchigarev <dmitry.chigarev@intel.com>
// RUN: gc-opt %s --gc-gpu-pipeline \
// RUN: | gc-cpu-runner -e main --entry-point-result=void \
// RUN: --shared-libs=%mlir_runner_utils,%mlir_c_runner_utils,%opencl_runtime | FileCheck %s
module{
Copy link
Contributor Author

Choose a reason for hiding this comment

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

changed the size of the test from 32x32 to 64x64 due to #332

Signed-off-by: dchigarev <dmitry.chigarev@intel.com>
@dchigarev dchigarev marked this pull request as ready for review September 11, 2024 08:11
Comment on lines +1413 to +1414
// Otherwise 'xegpu-to-vc' pass will fail to convert it to VC
if (outputShape[0] * outputShape[1] < 16) {
Copy link
Contributor Author

@dchigarev dchigarev Sep 11, 2024

Choose a reason for hiding this comment

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

gpu-runner/mlp.mlir test contains a linalg.fill() : tensor<1x1xf32> op which is to small to be processed via xegpu

@dchigarev dchigarev merged commit d9bbc85 into intel:main Sep 16, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants