https://github.com/PiotrZSL requested changes to this pull request.
for me oryginal version that used registerMatchers instead of visitor were way better, specially that there are matchers that could be used for this, or could be written. And even for performance reasons it could be implemented in slightly better way by simply very quickly exclude all classes that do not derive from std::enable_shared_from_this. If `boost::enable_shared_from_this` got same flaw, then this check should also support it. And this mean it maybe should be configurable. https://github.com/llvm/llvm-project/pull/102299 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
