diff options
| author | aldot <aldot@138bc75d-0d04-0410-961f-82ee72b054a4> | 2013-02-06 16:55:35 +0000 |
|---|---|---|
| committer | aldot <aldot@138bc75d-0d04-0410-961f-82ee72b054a4> | 2013-02-06 16:55:35 +0000 |
| commit | 72fb5d678993b6d9856113b025be7a1a7e1387be (patch) | |
| tree | 67ab083e3c60a2371bc0bd0a37fde1ac61bd40b0 /contrib | |
| parent | 8b1e0325b111be4be290c3b97ab9bd69cafec841 (diff) | |
| download | gcc-72fb5d678993b6d9856113b025be7a1a7e1387be.tar.gz | |
validate_failures.py: Fix performance regression
2013-02-06 Bernhard Reutner-Fischer <aldot@gcc.gnu.org>
* testsuite-management/validate_failures.py
(IsInterestingResult): Fix performance regression
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@195811 138bc75d-0d04-0410-961f-82ee72b054a4
Diffstat (limited to 'contrib')
| -rw-r--r-- | contrib/ChangeLog | 5 | ||||
| -rwxr-xr-x | contrib/testsuite-management/validate_failures.py | 3 |
2 files changed, 7 insertions, 1 deletions
diff --git a/contrib/ChangeLog b/contrib/ChangeLog index eaccdbdeee9..0ace63edcf0 100644 --- a/contrib/ChangeLog +++ b/contrib/ChangeLog @@ -1,3 +1,8 @@ +2013-02-06 Bernhard Reutner-Fischer <aldot@gcc.gnu.org> + + * testsuite-management/validate_failures.py + (IsInterestingResult): Fix performance regression + 2013-02-04 Richard Sandiford <rdsandiford@googlemail.com> * update-copyright.py: New file. diff --git a/contrib/testsuite-management/validate_failures.py b/contrib/testsuite-management/validate_failures.py index ec51de91b6b..5c80ca30be8 100755 --- a/contrib/testsuite-management/validate_failures.py +++ b/contrib/testsuite-management/validate_failures.py @@ -62,6 +62,7 @@ import sys # Handled test results. _VALID_TEST_RESULTS = [ 'FAIL', 'UNRESOLVED', 'XPASS', 'ERROR' ] +_VALID_TEST_RESULTS_REX = re.compile("%s" % "|".join(_VALID_TEST_RESULTS)) # Subdirectory of srcdir in which to find the manifest file. _MANIFEST_SUBDIR = 'contrib/testsuite-management' @@ -210,7 +211,7 @@ def IsInterestingResult(line): if '|' in line: (_, line) = line.split('|', 1) line = line.strip() - return any(line.startswith(result) for result in _VALID_TEST_RESULTS) + return bool(_VALID_TEST_RESULTS_REX.match(line)) def IsInclude(line): |