Skip to content
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

Topic/profiling #246

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Topic/profiling #246

wants to merge 4 commits into from

Conversation

flsobral
Copy link
Member

Description:

Describe your changes in details.

Related Issue:

  • If you suggest a new feature or change, discuss it first in an issue;
  • If fixing a bug, there should be an issue open describing it with the steps to reproduce.

Please link the issue here.

Motivation and Context:

Why is this change required? What problem does it solve?

If it fixes an open issue (different from the previously mentioned), please link to the issue here.

Benefited Devices:

  • Device: [e.g. Raspberry Pi 3 model b+]
  • OS: [e.g. Linux Raspbian 10]

How Has This Been Tested?

  • Please describe in detail how you tested your changes;
  • Include details of your testing environment, and the tests you ran to;
  • See how your change affects other areas of the code, etc.

Don't forget to tell us witch TotalCross version you are using.

Tested Devices:

  • Device: [e.g. Raspberry Pi 3 model b+]
  • OS: [e.g. Linux Raspbian 10]

Screenshots or videos:

If appropriate.

flsobral and others added 4 commits December 21, 2020 17:16
Signed-off-by: Farshid Monhaseri <[email protected]>
-The 'Wall-time' profiling functions & macros remained as they were
-The new functions just calculate the mean average time for each target
function and avoid some near zero values in the calculations.
-We now switch between the set of profiling functions by defining their
own macros

Signed-off-by: Farshid Monhaseri <[email protected]>
@acmlira acmlira mentioned this pull request Jan 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants