Skip to content

Conversation

@aykevl
Copy link
Member

@aykevl aykevl commented Apr 14, 2020

Be able to run tinygo gdb -target=arduino ./testdata/alias.go and debug a program with the power of a real debugger.

Note that this only works on LLVM 11 (see #1056) because older versions have a bug in the AVR backend that cause it to produce invalid debug information: https://reviews.llvm.org/D74213.

Screenshot from 2020-04-14 23-20-29

@niaow niaow added this to the v0.15 milestone Jun 27, 2020
@niaow niaow self-requested a review June 27, 2020 18:20
@aykevl
Copy link
Member Author

aykevl commented Oct 16, 2020

This PR is now ready. It depends on LLVM 11 to be usable, but that has been merged in #1056.

@deadprogram
Copy link
Member

Something went wrong with the CI with the LLVM 11 build @aykevl you should take a look. Thanks.

@aykevl
Copy link
Member Author

aykevl commented Oct 16, 2020

Looking into it now, I can't seem to reproduce it locally (only in CI).

@aykevl
Copy link
Member Author

aykevl commented Oct 16, 2020

Huh, this is triggered by Go 1.15.3.

@aykevl
Copy link
Member Author

aykevl commented Oct 16, 2020

Managed to find the cause, see: golang/go#42032.

Maybe we should just use Go 1.15.2 until it is fixed.

@aykevl
Copy link
Member Author

aykevl commented Oct 17, 2020

Pushed fix: #1443

@deadprogram
Copy link
Member

#1143 has been merged, @aykevl can you please rebase this PR to make sure it passes CI now?

@aykevl
Copy link
Member Author

aykevl commented Oct 17, 2020

Rebased.

Be able to run `tinygo gdb -target=arduino examples/serial` and debug a
program with the power of a real debugger.

Note that this only works on LLVM 11 because older versions have a bug
in the AVR backend that cause it to produce invalid debug information:
https://reviews.llvm.org/D74213.
@aykevl
Copy link
Member Author

aykevl commented Dec 10, 2020

Rebased again.

@deadprogram
Copy link
Member

🐛 glad to be finally merging, thanks @aykevl

@deadprogram deadprogram merged commit 4cc1cdf into dev Dec 10, 2020
@deadprogram deadprogram deleted the gdb-simavr branch December 10, 2020 15:44
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