Skip to content

Conversation

clementval
Copy link
Contributor

No description provided.

@clementval clementval requested a review from wangzpgi January 22, 2025 19:13
@llvmbot llvmbot added flang Flang issues not falling into any other category flang:fir-hlfir labels Jan 22, 2025
@llvmbot
Copy link
Member

llvmbot commented Jan 22, 2025

@llvm/pr-subscribers-flang-fir-hlfir

Author: Valentin Clement (バレンタイン クレメン) (clementval)

Changes

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

1 Files Affected:

  • (modified) flang/lib/Optimizer/Dialect/CUF/CUFToLLVMIRTranslation.cpp (+2-1)
diff --git a/flang/lib/Optimizer/Dialect/CUF/CUFToLLVMIRTranslation.cpp b/flang/lib/Optimizer/Dialect/CUF/CUFToLLVMIRTranslation.cpp
index 0c1424d11b515c..7ed7f355959683 100644
--- a/flang/lib/Optimizer/Dialect/CUF/CUFToLLVMIRTranslation.cpp
+++ b/flang/lib/Optimizer/Dialect/CUF/CUFToLLVMIRTranslation.cpp
@@ -71,7 +71,8 @@ LogicalResult registerKernel(cuf::RegisterKernelOp op,
   llvm::Function *fctSym =
       moduleTranslation.lookupFunction(op.getKernelName().str());
   if (!fctSym)
-    return op.emitError() << "Couldn't find kernel name symbol";
+    return op.emitError() << "Couldn't find kernel name symbol: "
+                          << op.getKernelName().str();
   builder.CreateCall(fct, {modulePtr, fctSym,
                            getOrCreateFunctionName(
                                module, builder, op.getKernelModuleName().str(),

@clementval clementval merged commit c6e7b4a into llvm:main Jan 22, 2025
11 checks passed
@clementval clementval deleted the cuf_error_name branch January 22, 2025 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flang:fir-hlfir flang Flang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants