With VC, fastMalloc.cpp doesn't use inline assembly for LgFloor(). #if defined(_MSC_VER) && !defined(_68K_) && !defined(_MPPC_) && !defined(_X86_) && !defined(_IA64_) && !defined(_AMD64_) && defined(_M_IX86) static inline int LgFloor(size_t n) { __asm xor eax,eax __asm not eax __asm bsr eax,n } #endif p.s. In WebKit.dll, do_malloc() and do_free() expands as ALWAYS_INLINE everywhere and costs binary sizes. Is this intended?
<rdar://problem/5335830>
Yes, we don't have all the inline assembly implemented for Windows yet. It could be a small performance boost once done.
Why are the xor and the not needed in the inline assembly here? They don't seem to be needed in the gcc version.
P5 is not used for WebKit bugs. [http://webkit.org/quality/bugpriorities.html]
I don't think this method has shown up under recent profiling, and this bug has sat idle for 5 years. I don't think this is a change we intend to make. Please reopen if you intend to work on this bug.