https://github.com/ldionne approved this pull request. LGTM. After verification, we already include `<vector>` in `<functional>` for transitive-includes stability, so this shouldn't have user impact. https://github.com/llvm/llvm-project/pull/144310