Various tool overhauls #60
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add option to skip python dependency installation (useful for environments without internet connection). - Fix CheckArchSupport cmake script for RISC-V - Add "-fno-stack-protector" flag to avoid generating segfaults on the support library when tracing. - Remove usage of SIGSTKSZ in memory.h, since this variable is no longer available at compile time starting with glibc 2.34
- Refactor code for chop-perf-invok to improve readability, addition of future features. - Add option to take samples at a fixed time interval. - Fix chop-perf-invok test error reporting
- chop-weight is a tool for calculating the weight of a function invocation in terms of representativeness (after clustering). A python script exists already for doing so (cti_micro_weight), but this tool is way faster (written and optimized in C). - chop-perf-avg is a tool for averaging multiple chop-perf-invok measurements, calculating geometric mean and standard deviation.
Changes get_invocation_in_cluster to correctly index the cluster and invocation set arrays.
Code now checks for the return values of fread and getline
This needs to be generalized at some point in the feature.
Missing recursive detection in perf-invok
Signed-off-by: Ramon Bertran Monfort <[email protected]>
Signed-off-by: Ramon Bertran Monfort <[email protected]>
Signed-off-by: Ramon Bertran Monfort <[email protected]>
Signed-off-by: Ramon Bertran Monfort <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Missing the implementation of dummy functions in the tracer, which is still WIP and contains too many hard-coded variables.