svn commit: r256085 - stable/9/contrib/llvm/lib/CodeGen/SelectionDAG

[Dimitry Andric]

Author: dim
Date: Sun Oct  6 12:39:12 2013
New Revision: 256085
URL: http://svnweb.freebsd.org/changeset/base/256085

Log:
  MFC r256024:
  
  Pull in r186338 from upstream llvm trunk:
  
    Remove invalid assert in DAGTypeLegalizer::RemapValue
  
    There is a comment at the top of DAGTypeLegalizer::PerformExpensiveChecks
    which, in part, says:
  
     // Note that these invariants may not hold momentarily when processing a node:
     // the node being processed may be put in a map before being marked Processed.
  
    Unfortunately, this assert would be valid only if the above-mentioned invariant
    held unconditionally. This was causing llc to assert when, in fact,
    everything was fine.
  
    Thanks to Richard Sandiford for investigating this issue!
  
    Fixes PR16562.
  
  This fixes assertions which could occur in the multimedia/ffmpeg1 and
  multimedia/ffmpeg2 ports.
  
  Reported by:	Matthias Apitz <guru at unixarea.de>

Modified:
  stable/9/contrib/llvm/lib/CodeGen/SelectionDAG/LegalizeTypes.cpp
Directory Properties:
  stable/9/contrib/llvm/   (props changed)

Modified: stable/9/contrib/llvm/lib/CodeGen/SelectionDAG/LegalizeTypes.cpp
==============================================================================
--- stable/9/contrib/llvm/lib/CodeGen/SelectionDAG/LegalizeTypes.cpp	Sun Oct  6 12:35:29 2013	(r256084)
+++ stable/9/contrib/llvm/lib/CodeGen/SelectionDAG/LegalizeTypes.cpp	Sun Oct  6 12:39:12 2013	(r256085)
@@ -615,7 +615,10 @@ void DAGTypeLegalizer::RemapValue(SDValu
     // replaced with other values.
     RemapValue(I->second);
     N = I->second;
-    assert(N.getNode()->getNodeId() != NewNode && "Mapped to new node!");
+
+    // Note that it is possible to have N.getNode()->getNodeId() == NewNode at
+    // this point because it is possible for a node to be put in the map before
+    // being processed.
   }
 }