PseudoSourceValue: Update comments and fix lowercase variable names. NFC.
authorAlex Lorenz <arphaman@gmail.com>
Tue, 11 Aug 2015 22:23:19 +0000 (22:23 +0000)
committerAlex Lorenz <arphaman@gmail.com>
Tue, 11 Aug 2015 22:23:19 +0000 (22:23 +0000)
This commit updates the documentation comments in PseudoSourceValue.cpp and
PseudoSourceValue.h based on the LLVM's documentation style. It also fixes
several instances of variable names that started with a lowercase letter.

This change is done in preparation for the changes to the pseudo source value
object management and to the PseudoSourceValue's class hierarchy.

llvm-svn: 244686

llvm/include/llvm/CodeGen/PseudoSourceValue.h
llvm/lib/CodeGen/PseudoSourceValue.cpp

index e97cea3..dff20a6 100644 (file)
 #include "llvm/IR/Value.h"
 
 namespace llvm {
+
 class MachineFrameInfo;
 class MachineMemOperand;
 class raw_ostream;
 
 raw_ostream &operator<<(raw_ostream &OS, const MachineMemOperand &MMO);
 
-/// PseudoSourceValue - Special value supplied for machine level alias
-/// analysis. It indicates that a memory access references the functions
-/// stack frame (e.g., a spill slot), below the stack frame (e.g., argument
-/// space), or constant pool.
+/// Special value supplied for machine level alias analysis. It indicates that
+/// a memory access references the functions stack frame (e.g., a spill slot),
+/// below the stack frame (e.g., argument space), or constant pool.
 class PseudoSourceValue {
 private:
   friend class MachineMemOperand; // For printCustom().
 
-  /// printCustom - Implement printing for PseudoSourceValue. This is called
-  /// from Value::print or Value's operator<<.
-  ///
+  /// Implement printing for PseudoSourceValue. This is called from
+  /// Value::print or Value's operator<<.
   virtual void printCustom(raw_ostream &O) const;
 
 public:
-  /// isFixed - Whether this is a FixedStackPseudoSourceValue.
-  bool isFixed;
+  /// Whether this is a FixedStackPseudoSourceValue.
+  bool IsFixed;
 
-  explicit PseudoSourceValue(bool isFixed = false);
+  explicit PseudoSourceValue(bool IsFixed = false);
 
   virtual ~PseudoSourceValue();
 
-  /// isConstant - Test whether the memory pointed to by this
-  /// PseudoSourceValue has a constant value.
-  ///
+  /// Test whether the memory pointed to by this PseudoSourceValue has a
+  /// constant value.
   virtual bool isConstant(const MachineFrameInfo *) const;
 
-  /// isAliased - Test whether the memory pointed to by this
-  /// PseudoSourceValue may also be pointed to by an LLVM IR Value.
+  /// Test whether the memory pointed to by this PseudoSourceValue may also be
+  /// pointed to by an LLVM IR Value.
   virtual bool isAliased(const MachineFrameInfo *) const;
 
-  /// mayAlias - Return true if the memory pointed to by this
-  /// PseudoSourceValue can ever alias an LLVM IR Value.
+  /// Return true if the memory pointed to by this PseudoSourceValue can ever
+  /// alias an LLVM IR Value.
   virtual bool mayAlias(const MachineFrameInfo *) const;
 
   /// A pseudo source value referencing a fixed stack frame entry,
@@ -79,21 +77,17 @@ public:
   static const PseudoSourceValue *getJumpTable();
 };
 
-/// FixedStackPseudoSourceValue - A specialized PseudoSourceValue
-/// for holding FixedStack values, which must include a frame
-/// index.
+/// A specialized PseudoSourceValue for holding FixedStack values, which must
+/// include a frame index.
 class FixedStackPseudoSourceValue : public PseudoSourceValue {
   const int FI;
 
 public:
-  explicit FixedStackPseudoSourceValue(int fi)
-      : PseudoSourceValue(true), FI(fi) {}
+  explicit FixedStackPseudoSourceValue(int FI)
+      : PseudoSourceValue(true), FI(FI) {}
 
-  /// classof - Methods for support type inquiry through isa, cast, and
-  /// dyn_cast:
-  ///
   static inline bool classof(const PseudoSourceValue *V) {
-    return V->isFixed == true;
+    return V->IsFixed == true;
   }
 
   bool isConstant(const MachineFrameInfo *MFI) const override;
@@ -106,6 +100,7 @@ public:
 
   int getFrameIndex() const { return FI; }
 };
-} // End llvm namespace
+
+} // end namespace llvm
 
 #endif
index 5c9bec9..f012776 100644 (file)
@@ -60,7 +60,7 @@ const PseudoSourceValue *PseudoSourceValue::getConstantPool() {
 static const char *const PSVNames[] = {"Stack", "GOT", "JumpTable",
                                        "ConstantPool"};
 
-PseudoSourceValue::PseudoSourceValue(bool isFixed) : isFixed(isFixed) {}
+PseudoSourceValue::PseudoSourceValue(bool IsFixed) : IsFixed(IsFixed) {}
 
 PseudoSourceValue::~PseudoSourceValue() {}