Elaborate a bit on the type unit and ODR conditional code.
authorEric Christopher <echristo@gmail.com>
Mon, 29 Jul 2013 22:24:32 +0000 (22:24 +0000)
committerEric Christopher <echristo@gmail.com>
Mon, 29 Jul 2013 22:24:32 +0000 (22:24 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@187385 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/AsmPrinter/DwarfDebug.cpp

index 7ba1fc187da1cc29e0122924707a722e22fee56c..33d18db034d1c275f65f47fa8ce9d9b4eeee15fb 100644 (file)
@@ -1098,14 +1098,14 @@ void DwarfDebug::finalizeModuleInfo() {
     TheCU->constructContainingTypeDIEs();
   }
 
-  // For types that we'd like to move to type units or ODR check go ahead
-  // and either move the types out or add the ODR attribute now.
+  // Split out type units and conditionally add an ODR tag to the split
+  // out type.
   // FIXME: Do type splitting.
   for (unsigned i = 0, e = TypeUnits.size(); i != e; ++i) {
     MD5 Hash;
     DIE *Die = TypeUnits[i];
-    // If we're in C++ and we want to generate the hash then go ahead and do
-    // that now.
+    // If we've requested ODR hashes, the current language is C++, and the type
+    // isn't located inside a C++ anonymous namespace then add the attribute now.
     if (GenerateODRHash &&
         CUMap.begin()->second->getLanguage() == dwarf::DW_LANG_C_plus_plus &&
         !isContainedInAnonNamespace(Die))