emeryberger / Hoard

The Hoard Memory Allocator: A Fast, Scalable, and Memory-efficient Malloc for Linux, Windows, and Mac.
http://www.hoard.org
Apache License 2.0
1.1k stars 128 forks source link

Please remove the -g flag in the GNUMakefile for non debug builds. #37

Closed sirilvk closed 6 years ago

sirilvk commented 6 years ago

Please remove the -g flag in the GNUMakefile for non debug builds. Could see a performance improvement without that. (also looks like a typo in section LINUX_GCC_x86_64_COMPILE ).

emeryberger commented 6 years ago

Generating debug symbols should have no effect on performance (-DNDEBUG disables assertions, which does have an impact); compiler optimizations should be unaffected.

What is the typo?

sirilvk commented 6 years ago

Thought it would have been a mistake since it was in all the build configs. Also surprisingly after removing -g on Ubuntu server 17.10 x64 build I got a slightly faster execution time for my application. However closing this issue.