Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

bradjc
Copy link
Contributor

@bradjc bradjc commented Jun 21, 2023

Pull Request Overview

This pull request makes it easier to read the function names when using make stack-analysis.

Before:

❯ make stack-analysis
microbit_v2
----------------------
   main stack frame:
         1784     main

   5 largest stack frames:
         2408     _ZN7cortexm24kernel_hardfault_arm_v7m17h88793f49ade3c395E
         1784     main
          896     _ZN13capsules_core15process_console31ProcessConsole$LT$_$C$A$C$C$GT$19create_state_buffer4llll17h0554a6e14ac86a27E
          832     _ZN95_$LT$kernel..process_standard..ProcessStandard$LT$C$GT$$u20$as$u20$kernel..process..Process$GT$18print_full_process17haf8c56e314683244E
          736     _ZN13capsules_core15process_console31ProcessConsole$LT$_$C$A$C$C$GT$11write_state17hf1a8799619718071E

After:

❯ make stack-analysis
microbit_v2
----------------------
   main stack frame:
         1784     main

   5 largest stack frames:
         2408     cortexm::kernel_hardfault_arm_v7m
         1784     main
          832     <kernel::process_standard::ProcessStandard<nrf52::chip::NRF52<nrf52833::interrupt_service::Nrf52833DefaultPeripherals>> as kernel::process::Process>::print_full_process
          736     <capsules_core::process_console::ProcessConsole<10, capsules_core::virtualizers::virtual_alarm::VirtualMuxAlarm<nrf5x::rtc::Rtc>, components::process_console::Capability>>::write_state
          672     <capsules_core::process_console::ProcessConsole<10, capsules_core::virtualizers::virtual_alarm::VirtualMuxAlarm<nrf5x::rtc::Rtc>, components::process_console::Capability> as kernel::hil::uart::TransmitClient>::transmitted_buffer

Testing Strategy

Running make

TODO or Help Wanted

n/a

Documentation Updated

  • Updated the relevant files in /docs, or no updates are required.

Formatting

  • Ran make prepush.

@hudson-ayers hudson-ayers added this pull request to the merge queue Jun 25, 2023
Merged via the queue into master with commit 4b90a84 Jun 25, 2023
@alistair23 alistair23 deleted the stack-analysis-demangle branch June 26, 2023 23:00
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.

4 participants