Skip to content

Commit

Permalink
ci: add AVR timers test
Browse files Browse the repository at this point in the history
Add the timers test because they now work correctly on AVR, probably as
a result of the reflect refactor: #2640

I've also updated a few of the other tests to indicate the new status
and why they don't work. It's no longer because of compiler errors, but
because of linker or runtime errors (which is at least some progress).
For example, I found that testdata/reflect.go works if you disable
`testAppendSlice` and increase the stack size.
  • Loading branch information
aykevl authored and deadprogram committed Feb 26, 2023
1 parent 4766217 commit 201592d
Showing 1 changed file with 6 additions and 9 deletions.
15 changes: 6 additions & 9 deletions main_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -180,28 +180,25 @@ func runPlatTests(options compileopts.Options, tests []string, t *testing.T) {
// Skip the ones that aren't.
switch name {
case "reflect.go":
// Reflect tests do not work due to type code issues.
// Reflect tests do not run correctly, probably because of the
// limited amount of memory.
continue

case "gc.go":
// Does not pass due to high mark false positive rate.
continue

case "json.go", "stdlib.go", "testing.go":
// Breaks interp.
// Too big for AVR. Doesn't fit in flash/RAM.
continue

case "math.go":
// Stuck somewhere, not sure what's happening.
// Needs newer picolibc version (for sqrt).
continue

case "cgo/":
// CGo does not work on AVR.
continue

case "timers.go":
// Doesn't compile:
// panic: compiler: could not store type code number inside interface type code
// CGo function pointers don't work on AVR (needs LLVM 16 and
// some compiler changes).
continue

default:
Expand Down

0 comments on commit 201592d

Please sign in to comment.