[PATCH] D93637: [libTooling] Add support for smart pointers to releveant Transformer `Stencil`s.

[Tom Lokovic via Phabricator via cfe-commits]

tdl-g added inline comments.


================
Comment at: clang/unittests/Tooling/StencilTest.cpp:273
+  std::string Snippet = R"cc(
+    Smart x;
+    x;
----------------
You're only testing the "QuacksLike" case.  I suspect you should have tests that validate the "KnownSmartPointers".

Admittedly, it's a bit redundant since the known smart pointers also QuackLike pointers.  Which, I guess, raises the question of why you have the hard-coded list of KnownSmartPointers if they are covered by the QuacksLike behavior and thus can't be meaningfully tested independently.

What do you think?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93637/new/

https://reviews.llvm.org/D93637