[llvm] r205281 - Remove FIXMEs. The scope of a Variable is always a lexical scope; there is

Adrian Prantl aprantl at apple.com
Mon Mar 31 20:50:01 PDT 2014


Author: adrian
Date: Mon Mar 31 22:50:01 2014
New Revision: 205281

URL: http://llvm.org/viewvc/llvm-project?rev=205281&view=rev
Log:
Remove FIXMEs. The scope of a Variable is always a lexical scope; there is
nothing to be gained from switching this over to a DIScopeRef.

Modified:
    llvm/trunk/include/llvm/IR/DebugInfo.h

Modified: llvm/trunk/include/llvm/IR/DebugInfo.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/IR/DebugInfo.h?rev=205281&r1=205280&r2=205281&view=diff
==============================================================================
--- llvm/trunk/include/llvm/IR/DebugInfo.h (original)
+++ llvm/trunk/include/llvm/IR/DebugInfo.h Mon Mar 31 22:50:01 2014
@@ -612,7 +612,6 @@ class DIGlobalVariable : public DIDescri
 public:
   explicit DIGlobalVariable(const MDNode *N = 0) : DIDescriptor(N) {}
 
-  // FIXME: use DIScopeRef for LTO type uniqueing.
   DIScope getContext() const { return getFieldAs<DIScope>(2); }
   StringRef getName() const { return getStringField(3); }
   StringRef getDisplayName() const { return getStringField(4); }
@@ -646,7 +645,6 @@ class DIVariable : public DIDescriptor {
 public:
   explicit DIVariable(const MDNode *N = 0) : DIDescriptor(N) {}
 
-  // FIXME: use DIScopeRef for LTO type uniqueing.
   DIScope getContext() const { return getFieldAs<DIScope>(1); }
   StringRef getName() const { return getStringField(2); }
   DIFile getFile() const { return getFieldAs<DIFile>(3); }





More information about the llvm-commits mailing list