-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
native profiling tools support #789
Conversation
That's different to the "usual" profiling method for RIOT using |
This enables the use of gprof. The -finstrument-functions have nothing to do with it. |
If you think this is too confusing I'll rename the make targets to |
works on my system with ccn-lite-client: ACK |
@OlegHahm What do you think? |
Actually - after looking into this a bit - I'd like to change this either way. gprof is not the only tool for the job and certainly not the best. I'll update the PR shortly. |
Changed it. |
rebased |
only works with Linux for now
The new target names are easier to remember / decipher: term term-cachegrind term-gprof term-valgrind all all-cachegrind all-gprof all-valgrind all-debug eval-gprof eval-cachegrind
rebased, restructured native-only targets |
@OlegHahm ping |
@OlegHahm ping |
ACK - works with gcc and O2 / O3 @LudwigOrtmann please squash....and merge I'm going to use this code in my thesis |
Found it to be working. |
native profiling tools support
Only works with Linux for now.
Usage:
gprof:
make -B clean all-gprof; make term-gprof; make eval-gprof
cachegrind:
make -B clean all-cachegrind; make term-cachegrind; make eval-cachegrind
The native-only targets are being restructured along with this PR. They are now more accessible:
term
term-cachegrind
term-gprof
term-valgrind
all
all-cachegrind
all-debug
all-gprof
all-valgrind
eval-gprof
eval-cachegrind
I'll add documentation in the wiki when this gets merged.
Fixes #579