Cosmetic: lengthen names and improve comments.

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

diff --git a/lib/CodeGen/SelectionDAG/InstrEmitter.cpp b/lib/CodeGen/SelectionDAG/InstrEmitter.cpp
index e6ad3630508dbb9e5b6b5f1661410f1f08e7715c..223704305b714ebde0086d4eff86b53612213f36 100644 (file)
--- a/lib/CodeGen/SelectionDAG/InstrEmitter.cpp
+++ b/lib/CodeGen/SelectionDAG/InstrEmitter.cpp
@@ -508,7 +508,7 @@ InstrEmitter::EmitDbgValue(SDNode *Node,
     return;
   if (!sd)
     return;
-  assert(sd->getKind() == SDDbgValue::SD);
+  assert(sd->getKind() == SDDbgValue::SDNODE);
   unsigned VReg = getVR(SDValue(sd->getSDNode(), sd->getResNo()), VRBaseMap);
   const TargetInstrDesc &II = TII->get(TargetOpcode::DBG_VALUE);
   DebugLoc DL = sd->getDebugLoc();
@@ -537,7 +537,7 @@ InstrEmitter::EmitDbgValue(SDDbgValue *sd,
   SDDbgValue::DbgValueKind kind = sd->getKind();
   DebugLoc DL = sd->getDebugLoc();
   MachineInstr* MI;
-  if (kind == SDDbgValue::CNST) {
+  if (kind == SDDbgValue::CONST) {
     Value *V = sd->getConst();
     if (ConstantInt *CI = dyn_cast<ConstantInt>(V)) {
       MI = BuildMI(*MF, DL, II).addImm(CI->getZExtValue()).
@@ -551,7 +551,7 @@ InstrEmitter::EmitDbgValue(SDDbgValue *sd,
       MI = BuildMI(*MF, DL, II).addReg(0U).
                                        addImm(Offset).addMetadata(mdPtr);
     }
-  } else if (kind == SDDbgValue::FX) {
+  } else if (kind == SDDbgValue::FRAMEIX) {
     unsigned FrameIx = sd->getFrameIx();
     // Stack address; this needs to be lowered in target-dependent fashion.
     // FIXME test that the target supports this somehow; if not emit Undef.

diff --git a/lib/CodeGen/SelectionDAG/SDDbgValue.h b/lib/CodeGen/SelectionDAG/SDDbgValue.h
index 1b7ba18afeacc04e1901a41449878c5b52addd6f..6cccfc01534197c3837a99236b0fb0ac3b670bd2 100644 (file)
--- a/lib/CodeGen/SelectionDAG/SDDbgValue.h
+++ b/lib/CodeGen/SelectionDAG/SDDbgValue.h
@@ -24,22 +24,21 @@ class SDNode;
 class Value;
 
 /// SDDbgValue - Holds the information from a dbg_value node through SDISel.
-/// Either Const or Node is nonzero, but not both.
 /// We do not use SDValue here to avoid including its header.
 
 class SDDbgValue {
 public:
   enum DbgValueKind {
-    SD = 0,
-    CNST = 1,
-    FX = 2
+    SDNODE = 0,             // value is the result of an expression
+    CONST = 1,              // value is a constant
+    FRAMEIX = 2             // value is contents of a stack location
   };
 private:
   enum DbgValueKind kind;
   union {
     struct {
-      SDNode *Node;         // valid for non-constants
-      unsigned ResNo;       // valid for non-constants
+      SDNode *Node;         // valid for expressions
+      unsigned ResNo;       // valid for expressions
     } s;
     Value *Const;           // valid for constants
     unsigned FrameIx;       // valid for stack objects
@@ -52,7 +51,7 @@ public:
   // Constructor for non-constants.
   SDDbgValue(MDNode *mdP, SDNode *N, unsigned R, uint64_t off, DebugLoc dl,
              unsigned O) : mdPtr(mdP), Offset(off), DL(dl), Order(O) {
-    kind = SD;
+    kind = SDNODE;
     u.s.Node = N;
     u.s.ResNo = R;
   }
@@ -60,14 +59,14 @@ public:
   // Constructor for constants.
   SDDbgValue(MDNode *mdP, Value *C, uint64_t off, DebugLoc dl, unsigned O) : 
     mdPtr(mdP), Offset(off), DL(dl), Order(O) {
-    kind = CNST;
+    kind = CONST;
     u.Const = C;
   }
 
   // Constructor for frame indices.
   SDDbgValue(MDNode *mdP, unsigned FI, uint64_t off, DebugLoc dl, unsigned O) : 
     mdPtr(mdP), Offset(off), DL(dl), Order(O) {
-    kind = FX;
+    kind = FRAMEIX;
     u.FrameIx = FI;
   }
 
@@ -78,16 +77,16 @@ public:
   MDNode *getMDPtr() { return mdPtr; }
 
   // Returns the SDNode* for a register ref
-  SDNode *getSDNode() { assert (kind==SD); return u.s.Node; }
+  SDNode *getSDNode() { assert (kind==SDNODE); return u.s.Node; }
 
   // Returns the ResNo for a register ref
-  unsigned getResNo() { assert (kind==SD); return u.s.ResNo; }
+  unsigned getResNo() { assert (kind==SDNODE); return u.s.ResNo; }
 
   // Returns the Value* for a constant
-  Value *getConst() { assert (kind==CNST); return u.Const; }
+  Value *getConst() { assert (kind==CONST); return u.Const; }
 
   // Returns the FrameIx for a stack object
-  unsigned getFrameIx() { assert (kind==FX); return u.FrameIx; }
+  unsigned getFrameIx() { assert (kind==FRAMEIX); return u.FrameIx; }
 
   // Returns the offset.
   uint64_t getOffset() { return Offset; }