Skip to content

Comments

Align with new LLVM version used in GC#2

Merged
Menooker merged 2 commits intoMenooker:devfrom
dchigarev:update-llvm
Aug 2, 2024
Merged

Align with new LLVM version used in GC#2
Menooker merged 2 commits intoMenooker:devfrom
dchigarev:update-llvm

Conversation

@dchigarev
Copy link

@dchigarev dchigarev commented Aug 1, 2024

This PR aligns IMEX with the new LLVM version in GC (introduced here: intel/graph-compiler#204)

The update has broken the following tests in IMEX (namely this assert statement now fails), I'm not sure what exactly caused this problem (my changes to IMEX or new LLVM versions). We don't use XeTile in our linalg->xegpu->gpu exe pipeline, so our flow is not affected by this problem.

Failed Tests (28):
  IMEX :: Conversion/XeTileToXeGPU/gemm_preop.mlir
  IMEX :: Conversion/XeTileToXeGPU/sg_mixed_scf.mlir
  IMEX :: Conversion/XeTileToXeGPU/sg_scf_for.mlir
  IMEX :: Conversion/XeTileToXeGPU/sg_tiled_scf_for.mlir
  IMEX :: Integration/Dialect/XeTile/sg_gemm_1kx1kx1k_bf16_bf16_f32.mlir
  IMEX :: Integration/Dialect/XeTile/sg_gemm_1kx1kx1k_f16_f16_f16_blk_16x32x32.mlir
  IMEX :: Integration/Dialect/XeTile/sg_gemm_1kx1kx1k_f16_f16_f32.mlir
  IMEX :: Integration/Dialect/XeTile/sg_gemm_1kx1kx1k_f16_f16_f32_transpose_a.mlir
  IMEX :: Integration/Dialect/XeTile/sg_gemm_1kx1kx1k_f16_f16_f32_transpose_b.mlir
  IMEX :: Integration/Dialect/XeTile/sg_gemm_1kx1kx1k_preop_postop_bf16_bf16_f32.mlir
  IMEX :: Integration/Dialect/XeTile/sg_gemm_1kx1kx1k_transposed_b_preop_postop_bf16_bf16_f32.mlir
  IMEX :: Integration/Dialect/XeTile/sg_gemm_postop.mlir
  IMEX :: Integration/Dialect/XeTile/sg_gemm_pre_broadcast_a.mlir
  IMEX :: Integration/Dialect/XeTile/sg_gemm_pre_broadcast_b.mlir
  IMEX :: Integration/Dialect/XeTile/sg_gemm_preop_a.mlir
  IMEX :: Integration/Dialect/XeTile/sg_gemm_preop_a_b.mlir
  IMEX :: Integration/Dialect/XeTile/sg_gemm_preop_b.mlir

Signed-off-by: dchigarev <dmitry.chigarev@intel.com>
Signed-off-by: dchigarev <dmitry.chigarev@intel.com>
@@ -1,28 +0,0 @@
From 283951f026428a3c34b8a2b8f2498d55faf590f5 Mon Sep 17 00:00:00 2001
Copy link
Author

Choose a reason for hiding this comment

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

this patch is no longer appliable (seems that this has been fixed in upstream)

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.

2 participants