[PATCH] D12535: [PR24643] Speeding up FoldingSetNodeID::AddPointer

[Ben Craig via llvm-commits]

bcraig added a comment.

I am building with Visual Studio 2013, targeting x86 (not x64).
I am also building with clang 3.4, targeting Linux x64.

The memmove call was only on the Win32 builds, and it was definitely a result of the append call.  It was the top 'inclusive time' function for the toy program that I was profiling.  MSVC's std::uninitialized_copy eventually turns into memmove.

I will dig in a bit more and see if my typical pointers are small.


http://reviews.llvm.org/D12535