Add gprof profiling support. #7547
Conversation
|
I have updated the title description according to the recent changes. |
Rossmaxx
changed the title
|
I was wondering if it makes sense to allow gprof even in Debug mode. At least, it turned out that the results were not so far from the Release mode results. Still, I wonder if we should print a warning if gprof is active in Debug mode, so the user knows that the results may not be representative? |
|
I'll merge this in 2 days if no one objects. I choose to merge this with 1 approval because this is a trivial PR and shouldn't have any impact other than the flag |
Still, I wonder if we should print a warning if gprof is active in Debug mode, so the user knows that the results may not be representative? saw this now after I took another look. I could but I choose not to for now. I don't see any need to add a warning, building on debug mode, one should naturally know that the performance won't be good. |
CMakeLists.txt
Outdated
| @@ -664,6 +665,20 @@ elseif(MSVC) | |||
| add_compile_options("/utf-8") | |||
| ENDIF() | |||
|
|
|||
| # gcc builds support gprof for profiling | |||
| if(NOT CMAKE_CXX_COMPILER_ID MATCHES "GNU") | |||
There was a problem hiding this comment.
It's supported by Clang too: https://clang.llvm.org/docs/ClangCommandLineReference.html#cmdoption-clang-pg.
There was a problem hiding this comment.
should modifying the matches string to "GNU|CLANG" suffice or is there any other syntax which I should be looking at?
There was a problem hiding this comment.
I have made the change for now, if it's wrong, just follow up on it and i'll change it.
Adds gprof support for profiling the builds. Also removed the
-fomit-frame-pointerflag from swh and tap suite as the flag interferes with the-pgflag used here.per https://stackoverflow.com/questions/14666665/trying-to-understand-gcc-option-fomit-frame-pointer , this option makes getting stack traces nearly impossible, and the performance improvements seems to be negligible on 64 bit systems, especially when debugging.
to use gprof:
after building with the
WANT_DEBUG_GPROFflag turned on, run lmms, and after running,gprof ./lmms gmon.out > report.txtfrom the build directory. This will generate a text file with the profiling data. Do note that this method is tested only on GCC, and windows builds might not be supported.