search

pcwalton / gperftools

Fork of Google's performance tools for better profiling on ARM
BSD 3-Clause "New" or "Revised" License
2 stars 0 forks source link

compile error #1

Open doudouOUC opened 8 years ago

doudouOUC commented 8 years ago

When I compile this gperftools, I got the error.less gelf.h.So, Do I need the libgelf?

MarkoFilipovic commented 8 years ago

I experienced the same issue when trying to make this run on my specific arm-based system. So I downloaded the most recent libelf (0.8.13), have compiled it and then tried again to compile gperftools. Gperftools complained in src/stacktrace_arm-mcternan-inl.h, when #including gelf.h. So I commented out the line #include and gperftools built successfully. A quick test on the target hardware produced also some meningful results, so it looks it works for me. Since gelf.h contains only declarations, removing it seems all right to me. I wonder though whether this is really safe.

kai-unger commented 8 years ago

The same here when trying to compile for 32-bit ARM.

Marko's patch is promising. Would be nice if a real expert could please comment on whether it is safe to simply remove the include of gelf.h.