[llvm] [memprof] Use std::move in toMemProfRecord (PR #93133)

Snehasish Kumar via llvm-commits llvm-commits at lists.llvm.org
Thu May 23 11:57:10 PDT 2024


snehasish wrote:

> `AI` here is a local variable. This patch does not modify the current `IndexedMemProfRecord` in any way. In fact, `toMemProfRecord` is a const function.

Thanks for pointing out local variable. Clearly I didn't spend much time reading the rest of the function. Can you try out the reserve calls and commit this along with it?



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


More information about the llvm-commits mailing list