[PATCH] D100378: [AST] Use IntrusiveRefCntPtr for Introspection LocationCall.

[Nathan James via Phabricator via cfe-commits]

njames93 added inline comments.


================
Comment at: clang/lib/Tooling/NodeIntrospection.cpp:60
+  if (LHS.first == RHS.first)
+    return LHS.second->name() < RHS.second->name();
+  return LHS.first < RHS.first;
----------------
This is a slight change in behaviour, The old implementation used the operator< on a `std::shared_ptr`.
That in turns boils down to a comparison on the raw pointer, which is less than ideal.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100378