make fulltest: no -D040 and no --gc-debug with -runcore=fast,-O1,-O2 #1086

Closed
rurban opened this Issue Jul 21, 2014 · 1 comment

Comments

Projects
None yet
1 participant
@rurban
Member

rurban commented Jul 21, 2014

Do not switch to the gc-debug runcore when we want to test the fast runcore.
Ditto do not enable -D040 "fill I, N registers with garbage" with -O1 or -O2 optimizations.

benchmarks should also omit -D040 and --gc-debug.

Current errors with these changes in branch smoke-me/testfast-gh1086, because these are bad tests previously skipped based on a changed $ENV{TEST_PROG_ARGS} regexp.

testO1__t/op/calling.t                           (Wstat: 256 Tests: 104 Failed: 1)
  Failed test:  63
  Non-zero exit status: 1
testO2__t/compilers/imcc/syn/regressions.t       (Wstat: 256 Tests: 21 Failed: 1)
  Failed test:  2
  Non-zero exit status: 1
testO2__t/op/calling.t                           (Wstat: 256 Tests: 104 Failed: 1)
  Failed test:  63
  Non-zero exit status: 1

@rurban rurban added the Step-test label Jul 21, 2014

@rurban rurban self-assigned this Jul 21, 2014

rurban pushed a commit that referenced this issue Jul 21, 2014

Reini Urban
[test] no gc-debug runcore when testing the fast runcore
With make fulltest and variants (t/fullharness) we omit -D040 and --gc-debug
when testing -f -O1 -O2 and the various set_usual_target() tests.
This caught some -O1/-O2 errors and speeds up the fulltest suite.
We only need to run a full GC after each op and randomize the N/I regs with the
normal internal testsuite, but not the extensions and various other tests.

See [GH #1086]

Current errors:
  testO1__t/op/calling.t, testO2__t/op/calling.t,
  testO2__t/compilers/imcc/syn/regressions.t

rurban pushed a commit that referenced this issue Jul 21, 2014

Reini Urban
[test] adjust bad -O1/-O2 test regex
with the new fullharness settings there is no space before -O1 anymore.
Fixes GH #1086

rurban pushed a commit that referenced this issue Sep 6, 2014

Reini Urban
[test] no gc-debug runcore when testing the fast runcore
With make fulltest and variants (t/fullharness) we omit -D040 and --gc-debug
when testing -f -O1 -O2 and the various set_usual_target() tests.
This caught some -O1/-O2 errors and speeds up the fulltest suite.
We only need to run a full GC after each op and randomize the N/I regs with the
normal internal testsuite, but not the extensions and various other tests.

See [GH #1086]

Current errors:
  testO1__t/op/calling.t, testO2__t/op/calling.t,
  testO2__t/compilers/imcc/syn/regressions.t

rurban pushed a commit that referenced this issue Sep 6, 2014

Reini Urban
[test] adjust bad -O1/-O2 test regex
with the new fullharness settings there is no space before -O1 anymore.
Fixes GH #1086

rurban pushed a commit that referenced this issue Sep 6, 2014

Reini Urban
[test] no gc-debug runcore when testing the fast runcore
With make fulltest and variants (t/fullharness) we omit -D040 and --gc-debug
when testing -f -O1 -O2 and the various set_usual_target() tests.
This caught some -O1/-O2 errors and speeds up the fulltest suite.
We only need to run a full GC after each op and randomize the N/I regs with the
normal internal testsuite, but not the extensions and various other tests.

See [GH #1086]

Current errors:
  testO1__t/op/calling.t, testO2__t/op/calling.t,
  testO2__t/compilers/imcc/syn/regressions.t

rurban pushed a commit that referenced this issue Sep 6, 2014

Reini Urban
[test] adjust bad -O1/-O2 test regex
with the new fullharness settings there is no space before -O1 anymore.
Fixes GH #1086

@rurban rurban added this to the 6.9.0 milestone Oct 4, 2014

rurban pushed a commit that referenced this issue Oct 4, 2014

Reini Urban
[test] no gc-debug runcore when testing the fast runcore
With make fulltest and variants (t/fullharness) we omit -D040 and --gc-debug
when testing -f -O1 -O2 and the various set_usual_target() tests.
This caught some -O1/-O2 errors and speeds up the fulltest suite.
We only need to run a full GC after each op and randomize the N/I regs with the
normal internal testsuite, but not the extensions and various other tests.

See [GH #1086]

Current errors:
  testO1__t/op/calling.t, testO2__t/op/calling.t,
  testO2__t/compilers/imcc/syn/regressions.t

rurban pushed a commit that referenced this issue Oct 4, 2014

Reini Urban
[test] adjust bad -O1/-O2 test regex
with the new fullharness settings there is no space before -O1 anymore.
Fixes GH #1086

rurban pushed a commit that referenced this issue Oct 4, 2014

@rurban

This comment has been minimized.

Show comment
Hide comment
@rurban

rurban Oct 4, 2014

Member

Merged into 6.9.0 master

Member

rurban commented Oct 4, 2014

Merged into 6.9.0 master

@rurban rurban closed this Oct 4, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment