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

Mac debugging with LLDB #518

Merged
merged 1 commit into from
Apr 29, 2022
Merged

Mac debugging with LLDB #518

merged 1 commit into from
Apr 29, 2022

Conversation

neworderofjamie
Copy link
Contributor

While investigating #517, I wanted to use the debugger but no one had been tempted by the "good first issue" to fix #57. I now have and GeNN users on the Mac can now debug code generator bugs to their heart's content :D (this also must win some sort of "oldest bug fixed" prize as it was first raised in 2015!)

Fixes #57

@neworderofjamie neworderofjamie added this to the GeNN 4.7.1 milestone Apr 28, 2022
@neworderofjamie neworderofjamie changed the title Mac debugging with lldb Mac debugging with LLDB Apr 28, 2022
@codecov
Copy link

codecov bot commented Apr 28, 2022

Codecov Report

Merging #518 (d411dd9) into master (a53cbca) will decrease coverage by 7.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #518      +/-   ##
==========================================
- Coverage   86.77%   79.70%   -7.08%     
==========================================
  Files          84       79       -5     
  Lines       18081    14145    -3936     
==========================================
- Hits        15690    11274    -4416     
- Misses       2391     2871     +480     
Impacted Files Coverage Δ
src/genn/genn/code_generator/codeStream.cc 44.00% <0.00%> (-48.00%) ⬇️
include/genn/genn/logging.h 0.00% <0.00%> (-44.45%) ⬇️
include/genn/genn/code_generator/codeGenUtils.h 62.85% <0.00%> (-37.15%) ⬇️
...nn/genn/code_generator/synapseUpdateGroupMerged.cc 62.50% <0.00%> (-34.85%) ⬇️
src/genn/genn/code_generator/backendBase.cc 57.92% <0.00%> (-34.15%) ⬇️
...enn/genn/code_generator/neuronUpdateGroupMerged.cc 65.71% <0.00%> (-29.58%) ⬇️
src/genn/genn/code_generator/modelSpecMerged.cc 67.45% <0.00%> (-28.07%) ⬇️
src/genn/genn/code_generator/initGroupMerged.cc 70.83% <0.00%> (-24.31%) ⬇️
...enn/genn/code_generator/customUpdateGroupMerged.cc 74.03% <0.00%> (-20.71%) ⬇️
include/genn/genn/postsynapticModels.h 80.00% <0.00%> (-20.00%) ⬇️
... and 43 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a53cbca...d411dd9. Read the comment docs.

Copy link
Member

@tnowotny tnowotny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

haha ... the fix isn't that difficult or elaborate - suppose we could have done this a little bit earlier ;-)

@neworderofjamie neworderofjamie merged commit ebdcba0 into master Apr 29, 2022
@neworderofjamie neworderofjamie deleted the mac_debugging branch April 29, 2022 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Debugging on Mac
2 participants