r329151 - Fix typo in ASTStructuralEquivalence.cpp for UnaryTransform types.

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 3 23:31:21 PDT 2018


Author: ericwf
Date: Tue Apr  3 23:31:21 2018
New Revision: 329151

URL: http://llvm.org/viewvc/llvm-project?rev=329151&view=rev
Log:
Fix typo in ASTStructuralEquivalence.cpp for UnaryTransform types.

Previously UnaryTransformType nodes were comparing the same node
for structural equivalence. This was due to a typo where T1 was
on both sides of the comparison. This patch corrects that typo.

Unfortunately I couldn't find a way to test this change. It seems
that currently UnaryTransform nodes are never actually checked
for equivalence, only their canonical types are.

None the less, this correction seemed appropriate.

Modified:
    cfe/trunk/lib/AST/ASTStructuralEquivalence.cpp

Modified: cfe/trunk/lib/AST/ASTStructuralEquivalence.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/ASTStructuralEquivalence.cpp?rev=329151&r1=329150&r2=329151&view=diff
==============================================================================
--- cfe/trunk/lib/AST/ASTStructuralEquivalence.cpp (original)
+++ cfe/trunk/lib/AST/ASTStructuralEquivalence.cpp Tue Apr  3 23:31:21 2018
@@ -501,7 +501,7 @@ static bool IsStructurallyEquivalent(Str
   case Type::UnaryTransform:
     if (!IsStructurallyEquivalent(
             Context, cast<UnaryTransformType>(T1)->getUnderlyingType(),
-            cast<UnaryTransformType>(T1)->getUnderlyingType()))
+            cast<UnaryTransformType>(T2)->getUnderlyingType()))
       return false;
     break;
 




More information about the cfe-commits mailing list