[clang-tools-extra] [clang-tidy] Create bugprone-incorrect-enable-shared-from-this check (PR #102299)

Julian Schmidt via cfe-commits cfe-commits at lists.llvm.org
Sun Oct 13 09:34:51 PDT 2024


https://github.com/5chmidti commented:

This is 'basically' ready. Sorry for the delay.

---

As stated in the comment, I've benchmarked the RAV and matcher-based solution, and they perform the same. So the actual (performance) benefit of using the RAV does not actually exist. I'd suggest using the matcher based solution instead, as matchers should be the preferred tool in clang-tidy, and the implementation is much shorter/easier to understand at a glance.

---

I'd like to get a second approval in before we merge this.

https://github.com/llvm/llvm-project/pull/102299


More information about the cfe-commits mailing list