[clang] a49a41e - [AST][NFC] Update outdated comments in ASTStructuralEquivalence.cpp

Gabor Marton via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 21 06:55:14 PST 2020


Author: Gabor Marton
Date: 2020-02-21T15:54:58+01:00
New Revision: a49a41e7855fad426abf6ee9b8b88a535e8d33cc

URL: https://github.com/llvm/llvm-project/commit/a49a41e7855fad426abf6ee9b8b88a535e8d33cc
DIFF: https://github.com/llvm/llvm-project/commit/a49a41e7855fad426abf6ee9b8b88a535e8d33cc.diff

LOG: [AST][NFC] Update outdated comments in ASTStructuralEquivalence.cpp

Added: 
    

Modified: 
    clang/lib/AST/ASTStructuralEquivalence.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/AST/ASTStructuralEquivalence.cpp b/clang/lib/AST/ASTStructuralEquivalence.cpp
index 91a2f3a8391b..c29b7b2f5907 100644
--- a/clang/lib/AST/ASTStructuralEquivalence.cpp
+++ b/clang/lib/AST/ASTStructuralEquivalence.cpp
@@ -31,10 +31,9 @@
 // }
 // ```
 // Indeed, it has it's queue, which holds pairs of nodes, one from each graph,
-// this is the `DeclsToCheck` and it's pair is in `TentativeEquivalences`.
-// `TentativeEquivalences` also plays the role of the marking (`marked`)
-// functionality above, we use it to check whether we've already seen a pair of
-// nodes.
+// this is the `DeclsToCheck` member. `VisitedDecls` plays the role of the
+// marking (`marked`) functionality above, we use it to check whether we've
+// already seen a pair of nodes.
 //
 // We put in the elements into the queue only in the toplevel decl check
 // function:
@@ -57,11 +56,6 @@
 // doing. Thus, static implementation functions must not call the **member**
 // functions.
 //
-// So, now `TentativeEquivalences` plays two roles. It is used to store the
-// second half of the decls which we want to compare, plus it plays a role in
-// closing the recursion. On a long term, we could refactor structural
-// equivalency to be more alike to the traditional BFS.
-//
 //===----------------------------------------------------------------------===//
 
 #include "clang/AST/ASTStructuralEquivalence.h"


        


More information about the cfe-commits mailing list