Various cleanups

[DarkDimius]

Motivation

First commit is the result of profiling.
All other commits are results of running clang-tidy.

Test plan

CI

[jez]

Nice. The atomic increment is because we copy the TypeAndOrigins, it has a field that wraps a shared_ptr, and shared_ptrs copies require atomic increments?

[DarkDimius]

The atomic increment is because we copy the TypeAndOrigins, it has a field that wraps a shared_ptr, and shared_ptrs copies require atomic increments?

yes, and because we've defined a destructor it removed move assignment operator.