[llvm] [memprof] Use std::vector<Frame> instead of llvm::SmallVector<Frame> (NFC) (PR #94432)
Kazu Hirata via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 5 00:39:10 PDT 2024
kazutakahirata wrote:
> Consider `SmallVector<Frame, 0>`? `sizeof(SmallVector<X, 0>) < sizeof(vector<X>)`
That's super interesting. Thanks! I just tried `SmallVector<Frame, 0>`. The cycle count goes _up_ by 2.3% while the instruction goes down slightly by 0.09% (all relative to `SmallVector<Frame>`). My `llvm-profdata` is not built with ThinLTO, so maybe a certain call hierarchy is not collapsed!?
https://github.com/llvm/llvm-project/pull/94432
More information about the llvm-commits
mailing list