Fix typos in comments, NFC
authorRobin Morisset <morisset@google.com>
Fri, 29 Aug 2014 21:53:01 +0000 (21:53 +0000)
committerRobin Morisset <morisset@google.com>
Fri, 29 Aug 2014 21:53:01 +0000 (21:53 +0000)
Summary: Just fixing comments, no functional change.

Test Plan: N/A

Reviewers: jfb

Subscribers: mcrosier, llvm-commits

Differential Revision: http://reviews.llvm.org/D5130

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@216784 91177308-0d34-0410-b5e6-96231b3b80d8

25 files changed:
docs/ProgrammersManual.rst
include/llvm/Analysis/LazyCallGraph.h
include/llvm/Analysis/PostDominators.h
include/llvm/Analysis/TargetTransformInfo.h
include/llvm/CodeGen/MachinePostDominators.h
include/llvm/CodeGen/SelectionDAG.h
include/llvm/IR/IRBuilder.h
include/llvm/IR/Instructions.h
include/llvm/IR/Type.h
include/llvm/MC/MCAsmInfo.h
lib/Analysis/MemoryDependenceAnalysis.cpp
lib/Analysis/ScalarEvolutionNormalization.cpp
lib/AsmParser/LLParser.cpp
lib/CodeGen/AsmPrinter/DIEHash.cpp
lib/CodeGen/StackColoring.cpp
lib/ExecutionEngine/JIT/JITMemoryManager.cpp
lib/IR/DiagnosticPrinter.cpp
lib/Target/AArch64/AArch64ISelDAGToDAG.cpp
lib/Target/AArch64/AArch64ISelLowering.cpp
lib/Target/AArch64/AArch64ISelLowering.h
lib/Target/ARM/ARMBaseInstrInfo.cpp
lib/Target/Mips/Mips16InstrInfo.td
test/CodeGen/X86/2013-10-14-FastISel-incorrect-vreg.ll
test/MC/ELF/comdat.s
utils/TableGen/DAGISelMatcherOpt.cpp

index 447cadcacfcc8eb3123157e7fa45d69fd6d0b8c3..b06d2ebc976446b39d7a38d3bc3f2ecf5d1efa1e 100644 (file)
@@ -873,7 +873,7 @@ variety of customizations.
 llvm/ADT/ilist_node.h
 ^^^^^^^^^^^^^^^^^^^^^
 
-``ilist_node<T>`` implements the forward and backward links that are expected
+``ilist_node<T>`` implements the forward and backward links that are expected
 by the ``ilist<T>`` (and analogous containers) in the default manner.
 
 ``ilist_node<T>``\ s are meant to be embedded in the node type ``T``, usually
index 2828b042c4565db8d5503f5db539d1bf611bbf19..9a59844d6720eb7534def33d8743b490d9aef9ff 100644 (file)
@@ -537,7 +537,7 @@ public:
 
   static void *ID() { return (void *)&PassID; }
 
-  /// \brief Compute the \c LazyCallGraph for the module \c M.
+  /// \brief Compute the \c LazyCallGraph for the module \c M.
   ///
   /// This just builds the set of entry points to the call graph. The rest is
   /// built lazily as it is walked.
index d330755a0871e9c866090010da55e6529c2cad15..72cd35754c8a5739fb98c21db34ab5d35a17a516 100644 (file)
@@ -19,7 +19,7 @@
 namespace llvm {
 
 /// PostDominatorTree Class - Concrete subclass of DominatorTree that is used to
-/// compute the post-dominator tree.
+/// compute the post-dominator tree.
 ///
 struct PostDominatorTree : public FunctionPass {
   static char ID; // Pass identification, replacement for typeid
index 087ab302baebec8f970a1731e603595e07484531..75fcffc6e96a7cdba1fd437324f07126f4fca235 100644 (file)
@@ -183,7 +183,7 @@ public:
   /// should probably move to simpler cost metrics using the above.
   /// Alternatively, we could split the cost interface into distinct code-size
   /// and execution-speed costs. This would allow modelling the core of this
-  /// query more accurately as the a call is a single small instruction, but
+  /// query more accurately as a call is a single small instruction, but
   /// incurs significant execution cost.
   virtual bool isLoweredToCall(const Function *F) const;
 
index beb2c4f0c5c0e771b8c44d96d0e63182f72dae5c..aab5c407629f7405b389753763158b1e716ef2ad 100644 (file)
@@ -22,7 +22,7 @@ namespace llvm {
 
 ///
 /// PostDominatorTree Class - Concrete subclass of DominatorTree that is used
-/// to compute the post-dominator tree.
+/// to compute the post-dominator tree.
 ///
 struct MachinePostDominatorTree : public MachineFunctionPass {
 private:
index f7d21f9c53d296b7bdf811821b7b4468fc060fc3..eaec6dcd1c10833aa68d2898591601ce75d60d81 100644 (file)
@@ -749,7 +749,7 @@ public:
                    SDValue SV, unsigned Align);
 
   /// getAtomicCmpSwap - Gets a node for an atomic cmpxchg op. There are two
-  /// valid Opcodes. ISD::ATOMIC_CMO_SWAP produces the value loaded and a
+  /// valid Opcodes. ISD::ATOMIC_CMO_SWAP produces the value loaded and a
   /// chain result. ISD::ATOMIC_CMP_SWAP_WITH_SUCCESS produces the value loaded,
   /// a success flag (initially i1), and a chain.
   SDValue getAtomicCmpSwap(unsigned Opcode, SDLoc dl, EVT MemVT, SDVTList VTs,
index da06efd2a112d3887050ac5dca74786c71d04702..c074be8ad4a5182b9a9d0d6e2c151b303be1344d 100644 (file)
@@ -446,7 +446,7 @@ private:
 /// The first template argument handles whether or not to preserve names in the
 /// final instruction output. This defaults to on.  The second template argument
 /// specifies a class to use for creating constants.  This defaults to creating
-/// minimally folded constants.  The fourth template argument allows clients to
+/// minimally folded constants.  The third template argument allows clients to
 /// specify custom insertion hooks that are called on every newly created
 /// insertion.
 template<bool preserveNames = true, typename T = ConstantFolder,
index f533ef5c37e52fd8e51dd1e3c54faabc7cae9cb6..c009f5dc8127dd2edfcb7d1bfb08897dcd716a09 100644 (file)
@@ -135,7 +135,7 @@ public:
     return getSubclassDataFromInstruction() & 32;
   }
 
-  /// \brief Specify whether this alloca is used to represent the arguments to
+  /// \brief Specify whether this alloca is used to represent the arguments to
   /// a call.
   void setUsedWithInAlloca(bool V) {
     setInstructionSubclassData((getSubclassDataFromInstruction() & ~32) |
index 13df2ca45a87d235e7afd55cb121e50092c3a3ee..a36fb0f9702e6c1a13dfa31ed7e149c7dd136f5f 100644 (file)
@@ -323,7 +323,7 @@ public:
   }
 
   /// getContainedType - This method is used to implement the type iterator
-  /// (defined a the end of the file).  For derived types, this returns the
+  /// (defined at the end of the file).  For derived types, this returns the
   /// types 'contained' in the derived type.
   ///
   Type *getContainedType(unsigned i) const {
index dc37ef3d8361ad3a39bf32bde6bec199eba950d8..a94331546c94f99d4f40c68199a08ed580776510 100644 (file)
@@ -87,7 +87,7 @@ protected:
   bool HasMachoTBSSDirective;
 
   /// True if the compiler should emit a ".reference .constructors_used" or
-  /// ".reference .destructors_used" directive after the static ctor/dtor
+  /// ".reference .destructors_used" directive after the static ctor/dtor
   /// list.  This directive is only emitted in Static relocation model.  Default
   /// is false.
   bool HasStaticCtorDtorReferenceInStaticMode;
index 93e64888049bd4ff1e9464cebd40770eff71af90..aef241840acdf567991bf84ce927eef1799cb5c0 100644 (file)
@@ -937,7 +937,7 @@ GetNonLocalInfoForBlock(const AliasAnalysis::Location &Loc,
   return Dep;
 }
 
-/// SortNonLocalDepInfoCache - Sort the NonLocalDepInfo cache, given a certain
+/// SortNonLocalDepInfoCache - Sort the NonLocalDepInfo cache, given a certain
 /// number of elements in the array that are already properly ordered.  This is
 /// optimized for the case when only a few entries are added.
 static void
index 3ccefb01101dcdc8d76bc7d7e64e958ac3f32255..b238fe43cc608a2b302f7d8b54fa555ee60e79d7 100644 (file)
@@ -126,7 +126,7 @@ TransformImpl(const SCEV *S, Instruction *User, Value *OperandValToReplace) {
       // Normalized form:   {-2,+,1,+,2}
       // Denormalized form: {1,+,3,+,2}
       //
-      // However, denormalization would use the a different step expression than
+      // However, denormalization would use a different step expression than
       // normalization (see getPostIncExpr), generating the wrong final
       // expression: {-2,+,1,+,2} + {1,+,2} => {-1,+,3,+,2}
       if (AR->isAffine() &&
index 7a06a48da76bd9dcc5f094ed272c00e2a95a7ad7..f363b550647a8c78c7e026b3ecc88d965e5227f0 100644 (file)
@@ -3337,7 +3337,7 @@ bool LLParser::ParseFunctionHeader(Function *&Fn, bool isDefine) {
   if (isDefine)
     return false;
 
-  // Check the declaration has no block address forward references.
+  // Check the declaration has no block address forward references.
   ValID ID;
   if (FunctionName.empty()) {
     ID.Kind = ValID::t_GlobalID;
index c2fad59aa4e17598f8ed1398115136dc21eca60f..b2a3ba88f94a7d01f56d5c339c09f17f1fcc2df0 100644 (file)
@@ -261,7 +261,7 @@ void DIEHash::hashDIEEntry(dwarf::Attribute Attribute, dwarf::Tag Tag,
     return;
   }
 
-  // otherwise, b) use the letter 'T' as the marker, ...
+  // otherwise, b) use the letter 'T' as the marker, ...
   addULEB128('T');
 
   addULEB128(Attribute);
index 370430c4c82a0968ea0c7762640dfdc497932fdc..dcf1b448a91059de9725f9104e06afeebaf35442 100644 (file)
@@ -228,7 +228,7 @@ void StackColoring::dump() const {
 unsigned StackColoring::collectMarkers(unsigned NumSlot) {
   unsigned MarkersFound = 0;
   // Scan the function to find all lifetime markers.
-  // NOTE: We use the a reverse-post-order iteration to ensure that we obtain a
+  // NOTE: We use a reverse-post-order iteration to ensure that we obtain a
   // deterministic numbering, and because we'll need a post-order iteration
   // later for solving the liveness dataflow problem.
   for (MachineBasicBlock *MBB : depth_first(MF)) {
index 0741f6ad94e63a7784efe2dd2216960486b01e10..2cd78b5a33a6a229c13f7a60ef607a0bc513ef5a 100644 (file)
@@ -680,7 +680,7 @@ sys::MemoryBlock DefaultJITMemoryManager::allocateNewSlab(size_t size) {
 bool DefaultJITMemoryManager::CheckInvariants(std::string &ErrorStr) {
   raw_string_ostream Err(ErrorStr);
 
-  // Construct the set of FreeRangeHeader pointers so we can query it
+  // Construct the set of FreeRangeHeader pointers so we can query it
   // efficiently.
   llvm::SmallPtrSet<MemoryRangeHeader*, 16> FreeHdrSet;
   FreeRangeHeader* FreeHead = FreeMemoryList;
index 5e160266c8fe7f8e1bbd42e98f7947a0d3e574af..f25fc20a197bf7ca618699838b4c105e4c3a013e 100644 (file)
@@ -7,7 +7,7 @@
 //
 //===----------------------------------------------------------------------===//
 //
-// This file defines the a diagnostic printer relying on raw_ostream.
+// This file defines a diagnostic printer relying on raw_ostream.
 //
 //===----------------------------------------------------------------------===//
 
index afe0504fb90ec590d31c5421d8eb4e2e93ca9a25..5a624e381ca291fe19fc29200a1bb919e00e7815 100644 (file)
@@ -303,7 +303,7 @@ static AArch64_AM::ShiftExtendType getShiftTypeForNode(SDValue N) {
 
 /// \brief Determine wether it is worth to fold V into an extended register.
 bool AArch64DAGToDAGISel::isWorthFolding(SDValue V) const {
-  // it hurts if the value is used at least twice, unless we are optimizing
+  // it hurts if the value is used at least twice, unless we are optimizing
   // for code size.
   if (ForCodeSize || V.hasOneUse())
     return true;
index f2054cb1dc0e4b0bbb93a53a65ef2afc43ccf155..05a6ade726229dbd32159d2a5a6f23a4f7a80712 100644 (file)
@@ -1781,8 +1781,7 @@ unsigned AArch64TargetLowering::getFunctionAlignment(const Function *F) const {
 
 #include "AArch64GenCallingConv.inc"
 
-/// Selects the correct CCAssignFn for a the given CallingConvention
-/// value.
+/// Selects the correct CCAssignFn for a given CallingConvention value.
 CCAssignFn *AArch64TargetLowering::CCAssignFnForCall(CallingConv::ID CC,
                                                      bool IsVarArg) const {
   switch (CC) {
index 4343ddc253d7c222448f77776a0cf4af54cb19bc..376430a2f67981073e48511b2a9a186e04b7474c 100644 (file)
@@ -199,8 +199,7 @@ class AArch64TargetLowering : public TargetLowering {
 public:
   explicit AArch64TargetLowering(TargetMachine &TM);
 
-  /// Selects the correct CCAssignFn for a the given CallingConvention
-  /// value.
+  /// Selects the correct CCAssignFn for a given CallingConvention value.
   CCAssignFn *CCAssignFnForCall(CallingConv::ID CC, bool IsVarArg) const;
 
   /// computeKnownBitsForTargetNode - Determine which of the bits specified in
index 0d38b2ab1995960bee9a73579e4432202d9e06d1..8ea87f9930c034ba845dfc77475551bce14f95b3 100644 (file)
@@ -2889,7 +2889,7 @@ static unsigned getNumMicroOpsSwiftLdSt(const InstrItineraryData *ItinData,
 // FIXME: The current MachineInstr design does not support relying on machine
 // mem operands to determine the width of a memory access. Instead, we expect
 // the target to provide this information based on the instruction opcode and
-// operands. However, using MachineMemOperand is the best solution now for
+// operands. However, using MachineMemOperand is the best solution now for
 // two reasons:
 //
 // 1) getNumMicroOps tries to infer LDM memory width from the total number of MI
index 5e4eebb62c1248b0198b71c7a1d3451c0f69ecc9..2364f4d7c336f661a1f1f6c85846d4ad89bed206 100644 (file)
@@ -1771,9 +1771,9 @@ def: Mips16Pat
 
 //
 // For constants, llvm transforms this to:
-// x > (k -1) and then reverses the operands to use setlt. So this pattern
+// x > (k - 1) and then reverses the operands to use setlt. So this pattern
 // is not used now by the compiler. (Presumably checking that k-1 does not
-// overflow). The compiler never uses this at the current time, due to
+// overflow). The compiler never uses this at the current time, due to
 // other optimizations.
 //
 //def: Mips16Pat
index bbba796eed2478353c6a3f33ee741c2b8818e755..10dc927200b629d9fe571a8d9ee5916cefb59c8b 100644 (file)
@@ -6,7 +6,7 @@
 ; we may reference variables that were not live across basic blocks
 ; resulting in undefined virtual registers.
 ;
-; In this example, this is illustrated by a the spill/reload of the
+; In this example, this is illustrated by a spill/reload of the
 ; LOADED_PTR_SLOT.
 ;
 ; Before this patch, the compiler was accessing two different spill
index 68b0f328f9bca437eda4c9bb6a6a7f725ba1e2e3..ae2cc6b228b7838b8f69f065d2107779db39a762 100644 (file)
@@ -1,6 +1,6 @@
 // RUN: llvm-mc -filetype=obj -triple x86_64-pc-linux-gnu %s -o - | llvm-readobj -s -t | FileCheck %s
 
-// Test that we produce the group sections and that they are a the beginning
+// Test that we produce the group sections and that they are at the beginning
 // of the file.
 
 // CHECK:        Section {
index da6a9577a7707110700eca20af25706ec261f76e..0c18e0b1c29add3b4833b8dd52f73bbdd4a66d46 100644 (file)
@@ -185,7 +185,7 @@ static void ContractNodes(std::unique_ptr<Matcher> &MatcherPtr,
 /// Conceptually, we'd like to sink these predicates all the way to the last
 /// matcher predicate in the series.  However, it turns out that some
 /// ComplexPatterns have side effects on the graph, so we really don't want to
-/// run a the complex pattern if the pattern predicate will fail.  For this
+/// run a complex pattern if the pattern predicate will fail.  For this
 /// reason, we refuse to sink the pattern predicate past a ComplexPattern.
 ///
 static void SinkPatternPredicates(std::unique_ptr<Matcher> &MatcherPtr) {