diff options
| author | Ayke van Laethem <[email protected]> | 2022-09-28 15:58:06 +0200 |
|---|---|---|
| committer | Ron Evans <[email protected]> | 2022-09-28 19:18:11 +0200 |
| commit | 895c54207612836c703f1f3744aac946e804bbb2 (patch) | |
| tree | c056aae4ed4dd17845eb7203f55821b7ec4ea958 /builder | |
| parent | e91fae57568df9d9f7073d9f32558f5ba4d9d3ff (diff) | |
| download | tinygo-895c54207612836c703f1f3744aac946e804bbb2.tar.gz tinygo-895c54207612836c703f1f3744aac946e804bbb2.zip | |
builder: do not ignore debug info on baremetal targets
Since https://github.com/tinygo-org/tinygo/pull/3200, `-no-debug` would
ignore debug info for some linkers. Example:
$ tinygo build -o test.elf -target=arduino -no-debug examples/blinky1
$ objdump -h test.elf
test.elf: file format elf32-avr
Sections:
Idx Name Size VMA LMA Type
0 00000000 00000000 00000000
1 .text 000004e0 00000000 00000000 TEXT
2 .trampolines 00000000 000004e0 000004e0 TEXT
3 .stack 00000200 00800100 00800100 BSS
4 .data 0000004c 00800300 000004e0 DATA
5 .bss 000000a9 0080034c 0000052c BSS
6 .debug_loc 00001bf0 00000000 00000000 DEBUG
7 .debug_abbrev 000004ed 00000000 00000000 DEBUG
8 .debug_info 00004176 00000000 00000000 DEBUG
9 .debug_ranges 00000150 00000000 00000000 DEBUG
10 .debug_str 0000206e 00000000 00000000 DEBUG
11 .debug_pubnames 000024bf 00000000 00000000 DEBUG
12 .debug_pubtypes 000004ca 00000000 00000000 DEBUG
13 .debug_line 0000193c 00000000 00000000 DEBUG
14 .debug_aranges 0000002c 00000000 00000000 DEBUG
15 .debug_rnglists 00000015 00000000 00000000 DEBUG
16 .debug_line_str 00000082 00000000 00000000 DEBUG
17 .shstrtab 000000d9 00000000 00000000
18 .symtab 000006d0 00000000 00000000
19 .strtab 00000607 00000000 00000000
This shows that even though `-no-debug` is supplied, debug information
is emitted in the ELF file.
With this change, debug information is not stripped when TinyGo doesn't
know how to do it:
$ tinygo build -o test.elf -target=arduino -no-debug examples/blinky1
error: cannot remove debug information: unknown linker: avr-gcc
(This issue will eventually be fixed by moving to `ld.lld`).
Diffstat (limited to 'builder')
| -rw-r--r-- | builder/build.go | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/builder/build.go b/builder/build.go index 5dfb5ea83..2008c897b 100644 --- a/builder/build.go +++ b/builder/build.go @@ -703,13 +703,6 @@ func Build(pkgName, outpath string, config *compileopts.Config, action func(Buil // Determine whether the compilation configuration would result in debug // (DWARF) information in the object files. var hasDebug = true - for _, tag := range config.BuildTags() { - if tag == "baremetal" { - // Don't use -no-debug on baremetal targets. It makes no sense: - // the debug information isn't flashed to the device anyway. - hasDebug = false - } - } if config.GOOS() == "darwin" { // Debug information isn't stored in the binary itself on MacOS but // is left in the object files by default. The binary does store the |