Commit e1fbcce8 authored by machenbach's avatar machenbach Committed by Commit bot

[foozzie] Fix suppressions and add one for crbug.com/681241

BUG=chromium:681241
NOTRY=true
TBR=bradnelson@chromium.org,titzer@chromium.org

Review-Url: https://codereview.chromium.org/2644503002
Cr-Commit-Position: refs/heads/master@{#42446}
parent 4064c41e
...@@ -33,28 +33,36 @@ MAX_LINE_LENGTH = 512 ...@@ -33,28 +33,36 @@ MAX_LINE_LENGTH = 512
# For ignoring lines before carets and to ignore caret positions. # For ignoring lines before carets and to ignore caret positions.
CARET_RE = re.compile(r'^\s*\^\s*$') CARET_RE = re.compile(r'^\s*\^\s*$')
# Ignore by original source files. Map from bug->relative file paths in V8, # Ignore by original source files. Map from bug->list of relative file paths in
# e.g. '/v8/test/mjsunit/d8-performance-now.js' including /v8/. A test will # V8, e.g. '/v8/test/mjsunit/d8-performance-now.js' including /v8/. A test will
# be suppressed if one of the files below was used to mutate the test. # be suppressed if one of the files below was used to mutate the test.
IGNORE_SOURCES = { IGNORE_SOURCES = {
# This contains a usage of f.arguments that often fires. # This contains a usage of f.arguments that often fires.
'crbug.com/662424': '/v8/test/mjsunit/regress/regress-2989.js', 'crbug.com/662424': ['/v8/test/mjsunit/regress/regress-2989.js'],
# crbug.com/681088 # crbug.com/681088
'crbug.com/681088': '/v8/test/mjsunit/asm/asm-validation.js', 'crbug.com/681088': [
'crbug.com/681088': '/v8/test/mjsunit/asm/b5528-comma.js', '/v8/test/mjsunit/asm/asm-validation.js',
'crbug.com/681088': '/v8/test/mjsunit/asm/pointer-masking.js', '/v8/test/mjsunit/asm/b5528-comma.js',
'crbug.com/681088': '/v8/test/mjsunit/compiler/regress-443744.js', '/v8/test/mjsunit/asm/pointer-masking.js',
'crbug.com/681088': '/v8/test/mjsunit/regress/regress-599719.js', '/v8/test/mjsunit/compiler/regress-443744.js',
'crbug.com/681088': '/v8/test/mjsunit/regress/wasm/regression-647649.js', '/v8/test/mjsunit/regress/regress-599719.js',
'crbug.com/681088': '/v8/test/mjsunit/wasm/asm-wasm.js', '/v8/test/mjsunit/regress/wasm/regression-647649.js',
'crbug.com/681088': '/v8/test/mjsunit/wasm/asm-wasm-deopt.js', '/v8/test/mjsunit/wasm/asm-wasm.js',
'crbug.com/681088': '/v8/test/mjsunit/wasm/asm-wasm-heap.js', '/v8/test/mjsunit/wasm/asm-wasm-deopt.js',
'crbug.com/681088': '/v8/test/mjsunit/wasm/asm-wasm-literals.js', '/v8/test/mjsunit/wasm/asm-wasm-heap.js',
'crbug.com/681088': '/v8/test/mjsunit/wasm/asm-wasm-stack.js', '/v8/test/mjsunit/wasm/asm-wasm-literals.js',
'/v8/test/mjsunit/wasm/asm-wasm-stack.js',
],
# crbug.com/681236 # crbug.com/681236
'crbug.com/681236': '/v8/test/mjsunit/wasm/asm-wasm-switch.js', 'crbug.com/681236': ['/v8/test/mjsunit/wasm/asm-wasm-switch.js'],
# crbug.com/681241
'crbug.com/681241': [
'/v8/test/mjsunit/regress/regress-617526.js',
'/v8/test/mjsunit/regress/wasm/regression-02862.js',
],
} }
# Ignore by test case pattern. Map from bug->regexp. # Ignore by test case pattern. Map from bug->regexp.
...@@ -300,9 +308,10 @@ class V8Suppression(Suppression): ...@@ -300,9 +308,10 @@ class V8Suppression(Suppression):
return False return False
def ignore_by_metadata(self, metadata): def ignore_by_metadata(self, metadata):
for bug, source in IGNORE_SOURCES.iteritems(): for bug, sources in IGNORE_SOURCES.iteritems():
if source in metadata['sources']: for source in sources:
return bug if source in metadata['sources']:
return bug
return False return False
def ignore_by_output1(self, output): def ignore_by_output1(self, output):
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment