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

gprof: Move gprof to libc #14683

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

gprof: Move gprof to libc #14683

wants to merge 2 commits into from

Conversation

W-M-R
Copy link
Contributor

@W-M-R W-M-R commented Nov 7, 2024

Summary

gprof: Move gprof to libc
1. Modify the name of the config
2. Since gprof is similar to gdbstub, this implementation is suitable for compilers such as gcc and clang, so it is more appropriate to put it in libc
gprof: Add necessary dependencies
when enable: CONFIG_SCHED_GPROF_ALL:
arm-none-eabi-g++: error: -pg and -fomit-frame-pointer are incompatible

Impact

None

Testing

config CONFIG_SCHED_GPROF,and CONFIG_SCHED_GPROF_ALL, CONFIG_SYSTEM_GPROF, run: gprof

@github-actions github-actions bot added Area: OS Components OS Components issues Size: XS The size of the change in this PR is very small labels Nov 7, 2024
@lupyuen
Copy link
Member

lupyuen commented Nov 7, 2024

Sorry I have to cancel the CI Check, there's a problem with CI Test:

when enable: CONFIG_SCHED_GPROF_ALL:
arm-none-eabi-g++: error: -pg and -fomit-frame-pointer are incompatible

Signed-off-by: wangmingrong1 <[email protected]>
@W-M-R W-M-R changed the title gprof: Add necessary dependencies gprof: Move gprof to libc Nov 7, 2024
@github-actions github-actions bot added Arch: arm Issues related to ARM (32-bit) architecture Arch: arm64 Issues related to ARM64 (64-bit) architecture Arch: simulator Issues related to the SIMulator Board: simulator Size: M The size of the change in this PR is medium and removed Size: XS The size of the change in this PR is very small labels Nov 7, 2024
1. Modify the name of the config
2. Since gprof is similar to gdbstub, this implementation is suitable for compilers such as gcc and clang, so it is more appropriate to put it in libc

Signed-off-by: wangmingrong1 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Arch: arm Issues related to ARM (32-bit) architecture Arch: arm64 Issues related to ARM64 (64-bit) architecture Arch: simulator Issues related to the SIMulator Area: OS Components OS Components issues Board: simulator Size: M The size of the change in this PR is medium
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants