Skip to content

Commit 4001fea

Browse files
committed
[CGSCC][LazyCallGraph][NFC] Fix typos in code comments
1 parent 5731381 commit 4001fea

File tree

2 files changed

+25
-24
lines changed

2 files changed

+25
-24
lines changed

llvm/include/llvm/Analysis/CGSCCPassManager.h

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
/// A secondary more general goal is to be able to isolate optimization on
2121
/// unrelated parts of the IR module. This is useful to ensure our
2222
/// optimizations are principled and don't miss oportunities where refinement
23-
/// of one part of the module influence transformations in another part of the
23+
/// of one part of the module influences transformations in another part of the
2424
/// module. But this is also useful if we want to parallelize the optimizations
2525
/// across common large module graph shapes which tend to be very wide and have
2626
/// large regions of unrelated cliques.
@@ -221,7 +221,7 @@ using ModuleAnalysisManagerCGSCCProxy =
221221
LazyCallGraph &>;
222222

223223
/// Support structure for SCC passes to communicate updates the call graph back
224-
/// to the CGSCC pass manager infrsatructure.
224+
/// to the CGSCC pass manager infrastructure.
225225
///
226226
/// The CGSCC pass manager runs SCC passes which are allowed to update the call
227227
/// graph and SCC structures. This means the structure the pass manager works
@@ -280,22 +280,22 @@ struct CGSCCUpdateResult {
280280

281281
/// If non-null, the updated current \c RefSCC being processed.
282282
///
283-
/// This is set when a graph refinement takes place an the "current" point in
284-
/// the graph moves "down" or earlier in the post-order walk. This will often
285-
/// cause the "current" RefSCC to be a newly created RefSCC object and the
286-
/// old one to be added to the above worklist. When that happens, this
283+
/// This is set when a graph refinement takes place and the "current" point
284+
/// in the graph moves "down" or earlier in the post-order walk. This will
285+
/// often cause the "current" RefSCC to be a newly created RefSCC object and
286+
/// the old one to be added to the above worklist. When that happens, this
287287
/// pointer is non-null and can be used to continue processing the "top" of
288288
/// the post-order walk.
289289
LazyCallGraph::RefSCC *UpdatedRC;
290290

291291
/// If non-null, the updated current \c SCC being processed.
292292
///
293-
/// This is set when a graph refinement takes place an the "current" point in
294-
/// the graph moves "down" or earlier in the post-order walk. This will often
295-
/// cause the "current" SCC to be a newly created SCC object and the old one
296-
/// to be added to the above worklist. When that happens, this pointer is
297-
/// non-null and can be used to continue processing the "top" of the
298-
/// post-order walk.
293+
/// This is set when a graph refinement takes place and the "current" point
294+
/// in the graph moves "down" or earlier in the post-order walk. This will
295+
/// often cause the "current" SCC to be a newly created SCC object and the
296+
/// old one to be added to the above worklist. When that happens, this
297+
/// pointer is non-null and can be used to continue processing the "top" of
298+
/// the post-order walk.
299299
LazyCallGraph::SCC *UpdatedC;
300300

301301
/// Preserved analyses across SCCs.
@@ -304,7 +304,7 @@ struct CGSCCUpdateResult {
304304
/// (changing both the CG structure and the function IR itself). However,
305305
/// this means we need to take special care to correctly mark what analyses
306306
/// are preserved *across* SCCs. We have to track this out-of-band here
307-
/// because within the main `PassManeger` infrastructure we need to mark
307+
/// because within the main `PassManager` infrastructure we need to mark
308308
/// everything within an SCC as preserved in order to avoid repeatedly
309309
/// invalidating the same analyses as we unnest pass managers and adaptors.
310310
/// So we track the cross-SCC version of the preserved analyses here from any

llvm/include/llvm/Analysis/LazyCallGraph.h

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ class LazyCallGraph {
145145
/// around but clear them.
146146
explicit operator bool() const;
147147

148-
/// Returnss the \c Kind of the edge.
148+
/// Returns the \c Kind of the edge.
149149
Kind getKind() const;
150150

151151
/// Test whether the edge represents a direct call to a function.
@@ -307,9 +307,9 @@ class LazyCallGraph {
307307

308308
/// A node in the call graph.
309309
///
310-
/// This represents a single node. It's primary roles are to cache the list of
311-
/// callees, de-duplicate and provide fast testing of whether a function is
312-
/// a callee, and facilitate iteration of child nodes in the graph.
310+
/// This represents a single node. Its primary roles are to cache the list of
311+
/// callees, de-duplicate and provide fast testing of whether a function is a
312+
/// callee, and facilitate iteration of child nodes in the graph.
313313
///
314314
/// The node works much like an optional in order to lazily populate the
315315
/// edges of each node. Until populated, there are no edges. Once populated,
@@ -392,7 +392,7 @@ class LazyCallGraph {
392392

393393
/// Internal helper to directly replace the function with a new one.
394394
///
395-
/// This is used to facilitate tranfsormations which need to replace the
395+
/// This is used to facilitate transformations which need to replace the
396396
/// formal Function object but directly move the body and users from one to
397397
/// the other.
398398
void replaceFunction(Function &NewF);
@@ -435,7 +435,7 @@ class LazyCallGraph {
435435
Nodes.clear();
436436
}
437437

438-
/// Print a short descrtiption useful for debugging or logging.
438+
/// Print a short description useful for debugging or logging.
439439
///
440440
/// We print the function names in the SCC wrapped in '()'s and skipping
441441
/// the middle functions if there are a large number.
@@ -467,9 +467,10 @@ class LazyCallGraph {
467467
/// Verify invariants about the SCC.
468468
///
469469
/// This will attempt to validate all of the basic invariants within an
470-
/// SCC, but not that it is a strongly connected componet per-se. Primarily
471-
/// useful while building and updating the graph to check that basic
472-
/// properties are in place rather than having inexplicable crashes later.
470+
/// SCC, but not that it is a strongly connected component per se.
471+
/// Primarily useful while building and updating the graph to check that
472+
/// basic properties are in place rather than having inexplicable crashes
473+
/// later.
473474
void verify();
474475
#endif
475476

@@ -511,7 +512,7 @@ class LazyCallGraph {
511512

512513
/// Provide a short name by printing this SCC to a std::string.
513514
///
514-
/// This copes with the fact that we don't have a name per-se for an SCC
515+
/// This copes with the fact that we don't have a name per se for an SCC
515516
/// while still making the use of this in debugging and logging useful.
516517
std::string getName() const {
517518
std::string Name;
@@ -644,7 +645,7 @@ class LazyCallGraph {
644645

645646
/// Provide a short name by printing this RefSCC to a std::string.
646647
///
647-
/// This copes with the fact that we don't have a name per-se for an RefSCC
648+
/// This copes with the fact that we don't have a name per se for an RefSCC
648649
/// while still making the use of this in debugging and logging useful.
649650
std::string getName() const {
650651
std::string Name;

0 commit comments

Comments
 (0)