[llvm] r231058 - Revert the non-cleanup part of r230769 because it introduced a non-determinism found only in the names of symbols.

Nick Lewycky nicholas at mxc.ca
Mon Mar 2 23:57:45 PST 2015


Author: nicholas
Date: Tue Mar  3 01:57:45 2015
New Revision: 231058

URL: http://llvm.org/viewvc/llvm-project?rev=231058&view=rev
Log:
Revert the non-cleanup part of r230769 because it introduced a non-determinism found only in the names of symbols.

Modified:
    llvm/trunk/utils/TableGen/CodeGenRegisters.cpp
    llvm/trunk/utils/TableGen/CodeGenRegisters.h

Modified: llvm/trunk/utils/TableGen/CodeGenRegisters.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/TableGen/CodeGenRegisters.cpp?rev=231058&r1=231057&r2=231058&view=diff
==============================================================================
--- llvm/trunk/utils/TableGen/CodeGenRegisters.cpp (original)
+++ llvm/trunk/utils/TableGen/CodeGenRegisters.cpp Tue Mar  3 01:57:45 2015
@@ -258,8 +258,6 @@ CodeGenRegister::computeSubRegs(CodeGenR
 
     // Look at the possible compositions of Idx.
     // They may not all be supported by SR.
-    // NOTE: Iteration order does not matter here because the EnumValue's
-    // of subreg indices are unique.
     for (CodeGenSubRegIndex::CompMap::const_iterator I = Comps.begin(),
            E = Comps.end(); I != E; ++I) {
       SubRegMap::const_iterator SRI = Map.find(I->first);

Modified: llvm/trunk/utils/TableGen/CodeGenRegisters.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/TableGen/CodeGenRegisters.h?rev=231058&r1=231057&r2=231058&view=diff
==============================================================================
--- llvm/trunk/utils/TableGen/CodeGenRegisters.h (original)
+++ llvm/trunk/utils/TableGen/CodeGenRegisters.h Tue Mar  3 01:57:45 2015
@@ -75,7 +75,8 @@ namespace llvm {
     std::string getQualifiedName() const;
 
     // Map of composite subreg indices.
-    typedef DenseMap<CodeGenSubRegIndex *, CodeGenSubRegIndex *> CompMap;
+    typedef std::map<CodeGenSubRegIndex *, CodeGenSubRegIndex *,
+                     deref<llvm::less>> CompMap;
 
     // Returns the subreg index that results from composing this with Idx.
     // Returns NULL if this and Idx don't compose.






More information about the llvm-commits mailing list