From 4e817b3fa36b0c8664a49af5c598f96efe8cfc18 Mon Sep 17 00:00:00 2001 From: Krzysztof Drewniak Date: Thu, 3 Mar 2022 22:52:27 +0000 Subject: [PATCH] [MLIR][AMDGPU] Fix typo and add comment to SerializeToHsaco Reviewed By: bondhugula Differential Revision: https://reviews.llvm.org/D120943 --- mlir/lib/Dialect/GPU/Transforms/SerializeToHsaco.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/mlir/lib/Dialect/GPU/Transforms/SerializeToHsaco.cpp b/mlir/lib/Dialect/GPU/Transforms/SerializeToHsaco.cpp index 36285ed..d33aa9d 100644 --- a/mlir/lib/Dialect/GPU/Transforms/SerializeToHsaco.cpp +++ b/mlir/lib/Dialect/GPU/Transforms/SerializeToHsaco.cpp @@ -239,6 +239,7 @@ SerializeToHsacoPass::translateToLLVMIR(llvm::LLVMContext &llvmContext) { constant->setAlignment(llvm::MaybeAlign(bitwidth / 8)); }; + // Set up control variables in the module instead of linking in tiny bitcode if (needOcml) { // TODO(kdrewnia): Enable math optimizations once we have support for // `-ffast-math`-like options @@ -277,7 +278,7 @@ SerializeToHsacoPass::translateToLLVMIR(llvm::LLVMContext &llvmContext) { if (!mbModules) { getOperation() - .emitWarning("Could not load required device labraries") + .emitWarning("Could not load required device libraries") .attachNote() << "This will probably cause link-time or run-time failures"; return ret; // We can still abort here -- 2.7.4