[SCEVExpander] Fix comments for functions. NFC.
authorSanjoy Das <sanjoy@playingwithpointers.com>
Tue, 14 Apr 2015 03:20:40 +0000 (03:20 +0000)
committerSanjoy Das <sanjoy@playingwithpointers.com>
Tue, 14 Apr 2015 03:20:40 +0000 (03:20 +0000)
Bring function documentation for ScalarEvolutionExpander up to code by
not repeating the function name in the comment documenting
functionality.  Reflow the edited comments where needed.

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

include/llvm/Analysis/ScalarEvolutionExpander.h

index 280a571e5c42fc8bea2f9768568b10635c260ae4..8ec2078258d111be7a78ee6bbfe48f13cbd7fd24 100644 (file)
@@ -28,7 +28,7 @@ namespace llvm {
   /// all materialized values are safe to speculate.
   bool isSafeToExpand(const SCEV *S, ScalarEvolution &SE);
 
-  /// SCEVExpander - This class uses information about analyze scalars to
+  /// This class uses information about analyze scalars to
   /// rewrite expressions in canonical form.
   ///
   /// Clients should create an instance of this class when rewriting is needed,
@@ -48,37 +48,36 @@ namespace llvm {
     std::set<AssertingVH<Value> > InsertedValues;
     std::set<AssertingVH<Value> > InsertedPostIncValues;
 
-    /// RelevantLoops - A memoization of the "relevant" loop for a given SCEV.
+    /// A memoization of the "relevant" loop for a given SCEV.
     DenseMap<const SCEV *, const Loop *> RelevantLoops;
 
-    /// PostIncLoops - Addrecs referring to any of the given loops are expanded
+    /// \brief Addrecs referring to any of the given loops are expanded
     /// in post-inc mode. For example, expanding {1,+,1}<L> in post-inc mode
     /// returns the add instruction that adds one to the phi for {0,+,1}<L>,
     /// as opposed to a new phi starting at 1. This is only supported in
     /// non-canonical mode.
     PostIncLoopSet PostIncLoops;
 
-    /// IVIncInsertPos - When this is non-null, addrecs expanded in the
-    /// loop it indicates should be inserted with increments at
-    /// IVIncInsertPos.
+    /// \brief When this is non-null, addrecs expanded in the loop it indicates
+    /// should be inserted with increments at IVIncInsertPos.
     const Loop *IVIncInsertLoop;
 
-    /// IVIncInsertPos - When expanding addrecs in the IVIncInsertLoop loop,
-    /// insert the IV increment at this position.
+    /// \brief When expanding addrecs in the IVIncInsertLoop loop, insert the IV
+    /// increment at this position.
     Instruction *IVIncInsertPos;
 
-    /// Phis that complete an IV chain. Reuse
+    /// \brief Phis that complete an IV chain. Reuse
     std::set<AssertingVH<PHINode> > ChainedPhis;
 
-    /// CanonicalMode - When true, expressions are expanded in "canonical"
-    /// form. In particular, addrecs are expanded as arithmetic based on
-    /// a canonical induction variable. When false, expression are expanded
-    /// in a more literal form.
+    /// \brief When true, expressions are expanded in "canonical" form. In
+    /// particular, addrecs are expanded as arithmetic based on a canonical
+    /// induction variable. When false, expression are expanded in a more
+    /// literal form.
     bool CanonicalMode;
 
-    /// When invoked from LSR, the expander is in "strength reduction" mode. The
-    /// only difference is that phi's are only reused if they are already in
-    /// "expanded" form.
+    /// \brief When invoked from LSR, the expander is in "strength reduction"
+    /// mode. The only difference is that phi's are only reused if they are
+    /// already in "expanded" form.
     bool LSRMode;
 
     typedef IRBuilder<true, TargetFolder> BuilderType;
@@ -91,7 +90,7 @@ namespace llvm {
     friend struct SCEVVisitor<SCEVExpander, Value*>;
 
   public:
-    /// SCEVExpander - Construct a SCEVExpander in "canonical" mode.
+    /// \brief Construct a SCEVExpander in "canonical" mode.
     explicit SCEVExpander(ScalarEvolution &se, const DataLayout &DL,
                           const char *name)
         : SE(se), DL(DL), IVName(name), IVIncInsertLoop(nullptr),
@@ -106,7 +105,7 @@ namespace llvm {
     void setDebugType(const char* s) { DebugType = s; }
 #endif
 
-    /// clear - Erase the contents of the InsertedExpressions map so that users
+    /// \brief Erase the contents of the InsertedExpressions map so that users
     /// trying to expand the same expression into multiple BasicBlocks or
     /// different places within the same BasicBlock can do so.
     void clear() {
@@ -116,38 +115,38 @@ namespace llvm {
       ChainedPhis.clear();
     }
 
-    /// isHighCostExpansion - Return true for expressions that may incur
-    /// non-trivial cost to evaluate at runtime.
+    /// \brief Return true for expressions that may incur non-trivial cost to
+    /// evaluate at runtime.
     bool isHighCostExpansion(const SCEV *Expr, Loop *L) {
       SmallPtrSet<const SCEV *, 8> Processed;
       return isHighCostExpansionHelper(Expr, L, Processed);
     }
 
-    /// getOrInsertCanonicalInductionVariable - This method returns the
-    /// canonical induction variable of the specified type for the specified
-    /// loop (inserting one if there is none).  A canonical induction variable
-    /// starts at zero and steps by one on each iteration.
+    /// \brief This method returns the canonical induction variable of the
+    /// specified type for the specified loop (inserting one if there is none).
+    /// A canonical induction variable starts at zero and steps by one on each
+    /// iteration.
     PHINode *getOrInsertCanonicalInductionVariable(const Loop *L, Type *Ty);
 
-    /// getIVIncOperand - Return the induction variable increment's IV operand.
+    /// \brief Return the induction variable increment's IV operand.
     Instruction *getIVIncOperand(Instruction *IncV, Instruction *InsertPos,
                                  bool allowScale);
 
-    /// hoistIVInc - Utility for hoisting an IV increment.
+    /// \brief Utility for hoisting an IV increment.
     bool hoistIVInc(Instruction *IncV, Instruction *InsertPos);
 
-    /// replaceCongruentIVs - replace congruent phis with their most canonical
+    /// \brief replace congruent phis with their most canonical
     /// representative. Return the number of phis eliminated.
     unsigned replaceCongruentIVs(Loop *L, const DominatorTree *DT,
                                  SmallVectorImpl<WeakVH> &DeadInsts,
                                  const TargetTransformInfo *TTI = nullptr);
 
-    /// expandCodeFor - Insert code to directly compute the specified SCEV
-    /// expression into the program.  The inserted code is inserted into the
-    /// specified block.
+    /// \brief Insert code to directly compute the specified SCEV expression
+    /// into the program.  The inserted code is inserted into the specified
+    /// block.
     Value *expandCodeFor(const SCEV *SH, Type *Ty, Instruction *I);
 
-    /// setIVIncInsertPos - Set the current IV increment loop and position.
+    /// \brief Set the current IV increment loop and position.
     void setIVIncInsertPos(const Loop *L, Instruction *Pos) {
       assert(!CanonicalMode &&
              "IV increment positions are not supported in CanonicalMode");
@@ -155,16 +154,15 @@ namespace llvm {
       IVIncInsertPos = Pos;
     }
 
-    /// setPostInc - Enable post-inc expansion for addrecs referring to the
-    /// given loops. Post-inc expansion is only supported in non-canonical
-    /// mode.
+    /// \brief Enable post-inc expansion for addrecs referring to the given
+    /// loops. Post-inc expansion is only supported in non-canonical mode.
     void setPostInc(const PostIncLoopSet &L) {
       assert(!CanonicalMode &&
              "Post-inc expansion is not supported in CanonicalMode");
       PostIncLoops = L;
     }
 
-    /// clearPostInc - Disable all post-inc expansion.
+    /// \brief Disable all post-inc expansion.
     void clearPostInc() {
       PostIncLoops.clear();
 
@@ -173,23 +171,22 @@ namespace llvm {
       InsertedPostIncValues.clear();
     }
 
-    /// disableCanonicalMode - Disable the behavior of expanding expressions in
-    /// canonical form rather than in a more literal form. Non-canonical mode
-    /// is useful for late optimization passes.
+    /// \brief Disable the behavior of expanding expressions in canonical form
+    /// rather than in a more literal form. Non-canonical mode is useful for
+    /// late optimization passes.
     void disableCanonicalMode() { CanonicalMode = false; }
 
     void enableLSRMode() { LSRMode = true; }
 
-    /// clearInsertPoint - Clear the current insertion point. This is useful
-    /// if the instruction that had been serving as the insertion point may
-    /// have been deleted.
+    /// \brief Clear the current insertion point. This is useful if the
+    /// instruction that had been serving as the insertion point may have been
+    /// deleted.
     void clearInsertPoint() {
       Builder.ClearInsertionPoint();
     }
 
-    /// isInsertedInstruction - Return true if the specified instruction was
-    /// inserted by the code rewriter.  If so, the client should not modify the
-    /// instruction.
+    /// \brief Return true if the specified instruction was inserted by the code
+    /// rewriter.  If so, the client should not modify the instruction.
     bool isInsertedInstruction(Instruction *I) const {
       return InsertedValues.count(I) || InsertedPostIncValues.count(I);
     }
@@ -199,29 +196,27 @@ namespace llvm {
   private:
     LLVMContext &getContext() const { return SE.getContext(); }
 
-    /// isHighCostExpansionHelper - Recursive helper function for
-    /// isHighCostExpansion.
+    /// \brief Recursive helper function for isHighCostExpansion.
     bool isHighCostExpansionHelper(const SCEV *S, Loop *L,
                                    SmallPtrSetImpl<const SCEV *> &Processed);
 
-    /// InsertBinop - Insert the specified binary operator, doing a small amount
+    /// \brief Insert the specified binary operator, doing a small amount
     /// of work to avoid inserting an obviously redundant operation.
     Value *InsertBinop(Instruction::BinaryOps Opcode, Value *LHS, Value *RHS);
 
-    /// ReuseOrCreateCast - Arange for there to be a cast of V to Ty at IP,
-    /// reusing an existing cast if a suitable one exists, moving an existing
-    /// cast if a suitable one exists but isn't in the right place, or
-    /// or creating a new one.
+    /// \brief Arrange for there to be a cast of V to Ty at IP, reusing an
+    /// existing cast if a suitable one exists, moving an existing cast if a
+    /// suitable one exists but isn't in the right place, or or creating a new
+    /// one.
     Value *ReuseOrCreateCast(Value *V, Type *Ty,
                              Instruction::CastOps Op,
                              BasicBlock::iterator IP);
 
-    /// InsertNoopCastOfTo - Insert a cast of V to the specified type,
-    /// which must be possible with a noop cast, doing what we can to
-    /// share the casts.
+    /// \brief Insert a cast of V to the specified type, which must be possible
+    /// with a noop cast, doing what we can to share the casts.
     Value *InsertNoopCastOfTo(Value *V, Type *Ty);
 
-    /// expandAddToGEP - Expand a SCEVAddExpr with a pointer type into a GEP
+    /// \brief Expand a SCEVAddExpr with a pointer type into a GEP
     /// instead of using ptrtoint+arithmetic+inttoptr.
     Value *expandAddToGEP(const SCEV *const *op_begin,
                           const SCEV *const *op_end,
@@ -229,13 +224,13 @@ namespace llvm {
 
     Value *expand(const SCEV *S);
 
-    /// expandCodeFor - Insert code to directly compute the specified SCEV
-    /// expression into the program.  The inserted code is inserted into the
-    /// SCEVExpander's current insertion point. If a type is specified, the
-    /// result will be expanded to have that type, with a cast if necessary.
+    /// \brief Insert code to directly compute the specified SCEV expression
+    /// into the program.  The inserted code is inserted into the SCEVExpander's
+    /// current insertion point. If a type is specified, the result will be
+    /// expanded to have that type, with a cast if necessary.
     Value *expandCodeFor(const SCEV *SH, Type *Ty = nullptr);
 
-    /// getRelevantLoop - Determine the most "relevant" loop for the given SCEV.
+    /// \brief Determine the most "relevant" loop for the given SCEV.
     const Loop *getRelevantLoop(const SCEV *);
 
     Value *visitConstant(const SCEVConstant *S) {