simark accepted this revision.
simark added inline comments.

================
Comment at: unittests/clangd/Printers.h:12
+// objects on failure. Otherwise, it is possible that there will be two
+// different instantiations of the printer template for a given type and some
+// tests could end up calling the wrong (default) one.
----------------
To be more precise, I think it's linker keeping only one of the two versions, 
so it's _all_ tests, not some, that may end up calling the wrong (default) one.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D44764



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

Reply via email to