[Mlir-commits] [mlir] [mlir][GPU] Refactor GPUOps lowering (PR #188905)

[Hocky Yudhiono]

================
@@ -74,6 +75,69 @@ LLVM::GlobalOp mlir::getOrCreateStringConstant(OpBuilder &b, Location loc,
                                 name, attr, alignment, addrSpace);
 }
 
+FailureOr<LowerLLVMFuncAttrs> GPUFuncOpLowering::buildLoweredGPULLVMFuncAttrs(
+    gpu::GPUFuncOp gpuFuncOp, Type llvmFuncType, OpBuilder &rewriter) const {
+  FailureOr<LowerLLVMFuncAttrs> loweredAttrs =
+      lowerDiscardableAttrsForLLVMFunc(gpuFuncOp);
+  if (failed(loweredAttrs))
+    return failure();
+
+  // TODO: How do we refactor this to be deny list?
+  NamedAttrList &discardable = loweredAttrs->discardableAttrs;
----------------
hockyy wrote:

> gpu.func is one of those ops that doesn't have its inherent attributes spelled out in rablegen properly, if I remember right. Might be worth fixing first.

got it, let me explore a bit and try to fix it.

https://github.com/llvm/llvm-project/pull/188905