[llvm] cece058 - [llvm][mlir][NFC] Fix typos in comments and test descriptions (#139688)

via llvm-commits llvm-commits at lists.llvm.org
Tue May 13 03:03:54 PDT 2025


Author: Wang Qiang
Date: 2025-05-13T11:03:51+01:00
New Revision: cece0581916b64cf67e7d4eecf3d1262d4382012

URL: https://github.com/llvm/llvm-project/commit/cece0581916b64cf67e7d4eecf3d1262d4382012
DIFF: https://github.com/llvm/llvm-project/commit/cece0581916b64cf67e7d4eecf3d1262d4382012.diff

LOG: [llvm][mlir][NFC] Fix typos in comments and test descriptions (#139688)

This patch fixes several typographical errors in comments and test
files:

1. Corrected "achive" to "archive" in archive-update.test. 
2. Fixed "achive" to "achieve" in a comment in
XeGPUSubgroupDistribute.cpp.
3. Corrected "achived" to "achieved" in a test note in
SimpleSIVNoValidityCheckFixedSize.ll.

These changes are non-functional and intended to improve readability and
documentation accuracy.

Signed-off-by: Kane Wang <wangqiang1 at kylinos.cn>
Co-authored-by: Kane Wang <wangqiang1 at kylinos.cn>

Added: 
    

Modified: 
    llvm/test/Analysis/DependenceAnalysis/SimpleSIVNoValidityCheckFixedSize.ll
    llvm/test/Object/archive-update.test
    mlir/lib/Dialect/XeGPU/Transforms/XeGPUSubgroupDistribute.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/test/Analysis/DependenceAnalysis/SimpleSIVNoValidityCheckFixedSize.ll b/llvm/test/Analysis/DependenceAnalysis/SimpleSIVNoValidityCheckFixedSize.ll
index c4948ebf9854b..2638bb045de3d 100644
--- a/llvm/test/Analysis/DependenceAnalysis/SimpleSIVNoValidityCheckFixedSize.ll
+++ b/llvm/test/Analysis/DependenceAnalysis/SimpleSIVNoValidityCheckFixedSize.ll
@@ -1,7 +1,7 @@
 ; NOTE: Assertions have been autogenerated by utils/update_analyze_test_checks.py UTC_ARGS: --version 5
 ; RUN: opt < %s -disable-output -passes="print<da>" 2>&1 | FileCheck %s
 
-; Note: exact results can be achived even if
+; Note: exact results can be achieved even if
 ; "-da-disable-delinearization-checks" is not used
 
 ;; #define N 1024

diff  --git a/llvm/test/Object/archive-update.test b/llvm/test/Object/archive-update.test
index e152da5705c16..7911c5808cdf2 100644
--- a/llvm/test/Object/archive-update.test
+++ b/llvm/test/Object/archive-update.test
@@ -17,7 +17,7 @@ RUN: touch -m -t 200001010000 %t/tmp.older/evenlen || sleep 2.1
 RUN: echo newer > %t/tmp.newer/evenlen
 RUN: touch %t/tmp.newer/evenlen
 
-Create an achive with the newest file
+Create an archive with the newest file
 RUN: llvm-ar rU %t/tmp.a %t/tmp.newer/evenlen
 RUN: llvm-ar p %t/tmp.a | FileCheck --check-prefix=NEWER %s
 

diff  --git a/mlir/lib/Dialect/XeGPU/Transforms/XeGPUSubgroupDistribute.cpp b/mlir/lib/Dialect/XeGPU/Transforms/XeGPUSubgroupDistribute.cpp
index acdbe7be1358b..992700524146a 100644
--- a/mlir/lib/Dialect/XeGPU/Transforms/XeGPUSubgroupDistribute.cpp
+++ b/mlir/lib/Dialect/XeGPU/Transforms/XeGPUSubgroupDistribute.cpp
@@ -835,7 +835,7 @@ LogicalResult LayoutAttrAssignment::run() {
 /// TODO: Implement the layout conflict resolution. This must ensure mainly two
 /// things:
 /// 1) Is a given layout supported by the op? (need to query the target
-///    HW info). Otherwise can we achive this layout using a layout conversion?
+///    HW info). Otherwise can we achieve this layout using a layout conversion?
 /// 2) Do all the operands have the required layout? If not, can it
 ///    be resolved using a layout conversion?
 LogicalResult LayoutAttrAssignment::resolveConflicts() { return success(); }


        


More information about the llvm-commits mailing list