Skip to content

Commit

Permalink
Reduce the size of tests for bug #1285 (#5561)
Browse files Browse the repository at this point in the history
As best I can tell the bug happens due to stuff happening _before_ the
user code. So I have removed anything after the user code (ie the giant
gob of dwarf debug info and symbols), in an attempt to reduce the test
sizes in general.

I manually undid the fix from #1285 and the tests correctly fail with
this reduced input.
  • Loading branch information
mattgodbolt committed Oct 8, 2023
1 parent d8d89c8 commit a94e106
Show file tree
Hide file tree
Showing 19 changed files with 1 addition and 209,649 deletions.
2 changes: 1 addition & 1 deletion test/filter-tests.js
Expand Up @@ -34,7 +34,7 @@ import {AsmParser} from '../lib/parsers/asm-parser.js';

import {fs, resolvePathFromTestRoot} from './utils.js';

approvals.mocha(resolvePathFromTestRoot('filter-cases'));
approvals.mocha(resolvePathFromTestRoot('filters-cases'));

function processAsm(filename, filters) {
const file = fs.readFileSync(filename, 'utf8');
Expand Down
12,171 changes: 0 additions & 12,171 deletions test/filters-cases/bug-1285.asm

Large diffs are not rendered by default.

0 comments on commit a94e106

Please sign in to comment.