[llvm] 5d9888f - [UniformityAnalysis] Miscellaneous comment fixes (#217661)

via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 21 02:48:30 PDT 2026


Author: Jay Foad
Date: 2026-08-21T09:48:25Z
New Revision: 5d9888f2d09f5f6e7dbfd157914f1c63180857cf

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

LOG: [UniformityAnalysis] Miscellaneous comment fixes (#217661)

Added: 
    

Modified: 
    llvm/include/llvm/ADT/GenericUniformityImpl.h

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/ADT/GenericUniformityImpl.h b/llvm/include/llvm/ADT/GenericUniformityImpl.h
index 9c8e421d3e5c5..c6f0bba051b60 100644
--- a/llvm/include/llvm/ADT/GenericUniformityImpl.h
+++ b/llvm/include/llvm/ADT/GenericUniformityImpl.h
@@ -15,6 +15,7 @@
 // This file should only be included by files that implement a
 // specialization of the relvant templates. Currently these are:
 // - UniformityAnalysis.cpp
+// - MachineUniformityAnalysis.cpp
 //
 // Note: The DEBUG_TYPE macro should be defined before using this
 // file so that any use of LLVM_DEBUG is associated with the
@@ -242,7 +243,7 @@ template <typename> class DivergencePropagator;
 //         x3 = phi
 //
 // The blocks D and F contain phi nodes and are thus each reachable
-// by two disjoins paths from A.
+// by two disjoint paths from A.
 //
 // -- Remarks --
 // * In case of cycle exits we need to check for temporal divergence.
@@ -256,7 +257,7 @@ template <typename> class DivergencePropagator;
 //   may have been produced by a nested irreducible cycle.
 //
 // * Note that SyncDependenceAnalysis is not concerned with the points
-//   of convergence in an irreducible cycle. It's only purpose is to
+//   of convergence in an irreducible cycle. Its only purpose is to
 //   identify join blocks. The "diverged entry" criterion is
 //   separately applied on join blocks to determine if an entire
 //   irreducible cycle is assumed to be divergent.
@@ -529,7 +530,7 @@ template <typename ContextT> class DivergencePropagator {
   const ContextT &Context;
 
   // Track blocks that receive a new label. Every time we relabel a
-  // cycle header, we another pass over the modified post-order in
+  // cycle header, we make another pass over the modified post-order in
   // order to propagate the header label. The bit vector also allows
   // us to skip labels that have not changed.
   SparseBitVector<> FreshLabels;
@@ -686,7 +687,7 @@ template <typename ContextT> class DivergencePropagator {
         break;
 
       const auto *Block = CyclePOT[BlockIdx];
-      // If no irreducible cycle, stop if freshLable.count() = 1 and Block
+      // If no irreducible cycle, stop if freshLabels.count() == 1 and Block
       // is the IPD. If it is in any irreducible cycle, continue propagation.
       if (FreshLabels.count() == 1 &&
           (!IrreducibleAncestor || !CI.contains(IrreducibleAncestor, Block)))


        


More information about the llvm-commits mailing list