Skip to content

Conversation

rupprecht
Copy link
Collaborator

This header uses GPUModuleOp but does not directly include the header: error: no type named 'GPUModuleOp' in namespace 'mlir::gpu'; did you mean 'ModuleOp'?

Needed for #148286

@rupprecht rupprecht added the skip-precommit-approval PR for CI feedback, not intended for review label Aug 14, 2025
@rupprecht rupprecht enabled auto-merge (squash) August 14, 2025 03:59
@llvmbot
Copy link
Member

llvmbot commented Aug 14, 2025

@llvm/pr-subscribers-mlir-llvm

@llvm/pr-subscribers-mlir

Author: Jordan Rupprecht (rupprecht)

Changes

This header uses GPUModuleOp but does not directly include the header: error: no type named 'GPUModuleOp' in namespace 'mlir::gpu'; did you mean 'ModuleOp'?

Needed for #148286


Full diff: https://github.com/llvm/llvm-project/pull/153532.diff

1 Files Affected:

  • (modified) mlir/include/mlir/Target/LLVM/XeVM/Utils.h (+1)
diff --git a/mlir/include/mlir/Target/LLVM/XeVM/Utils.h b/mlir/include/mlir/Target/LLVM/XeVM/Utils.h
index 32444de0cf90c..5d523f1ec457f 100644
--- a/mlir/include/mlir/Target/LLVM/XeVM/Utils.h
+++ b/mlir/include/mlir/Target/LLVM/XeVM/Utils.h
@@ -14,6 +14,7 @@
 #define MLIR_TARGET_LLVM_XEVM_UTILS_H
 
 #include "mlir/Dialect/GPU/IR/CompilationInterfaces.h"
+#include "mlir/Dialect/GPU/IR/GPUDialect.h"
 #include "mlir/Dialect/LLVMIR/XeVMDialect.h"
 #include "mlir/IR/Attributes.h"
 #include "mlir/Target/LLVM/ModuleToObject.h"

@rupprecht rupprecht merged commit 1d55b70 into llvm:main Aug 14, 2025
14 of 17 checks passed
@rupprecht rupprecht deleted the bzl-xe-parse-hdrs branch August 14, 2025 04:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mlir:llvm mlir skip-precommit-approval PR for CI feedback, not intended for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants