ABataev added inline comments.

================
Comment at: lib/AST/OpenMPClause.cpp:546
@@ +545,3 @@
+  for (auto *D : Declarations) {
+    if (Cache.count(D))
+      continue;
----------------
I think you'd better to count and to store canonical decls rather than the 
decls themselves. It will resolves issues for decls, where one decl is declared 
as an alias to another decl


http://reviews.llvm.org/D19382



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to