diff mbox

[GCC/contrib] Fix variant selection in dg-cmp-results.sh

Message ID a5de8bb9-c33d-e626-d195-cc30da378a38@foss.arm.com
State New
Headers show

Commit Message

Thomas Preudhomme June 21, 2017, 3:30 p.m. UTC 
Hi,

Commit r249422 to dg-cmp-results.sh broke the variant selection feature
where one can restrict the regression test to a specific target variant. This
fix restores the feature.


ChangeLog entry is as follows:

*** contrib/ChangeLog ***

2017-06-21  Thomas Preud'homme  <thomas.preudhomme@arm.com>

	* dg-cmp-results.sh: Restore filtering on target variant.


Tested on a file with multiple variants which now gives sane results.

Is this ok for trunk?

Best regards,

Thomas

Comments

Mike Stump June 21, 2017, 3:51 p.m. UTC | #1 
On Jun 21, 2017, at 8:30 AM, Thomas Preudhomme <thomas.preudhomme@foss.arm.com> wrote:
> 
> Commit r249422 to dg-cmp-results.sh broke the variant selection feature

> 2017-06-21  Thomas Preud'homme  <thomas.preudhomme@arm.com>
> 
> 	* dg-cmp-results.sh: Restore filtering on target variant.
> 
> 
> Tested on a file with multiple variants which now gives sane results.
> 
> Is this ok for trunk?

Ok.
diff mbox

Patch

diff --git a/contrib/dg-cmp-results.sh b/contrib/dg-cmp-results.sh
index 921e9337d1f8ffea78ef566c351fb48a8f6ca064..5f2fed5ec3ff0c66d22bc07c84571568730fbcac 100755
--- a/contrib/dg-cmp-results.sh
+++ b/contrib/dg-cmp-results.sh
@@ -90,7 +90,7 @@  echo "Newer log file: $NFILE"
 sed $E -e '/^[[:space:]]+===/,$d' $NFILE
 
 # Create a temporary file from the old file's interesting section.
-sed $E -e '/^Running target /,/^[[:space:]]+===.*Summary ===/!d' \
+sed $E -e "/$header/,/^[[:space:]]+===.*Summary ===/!d" \
   -e '/^[A-Z]+:/!d' \
   -e '/^(WARNING|ERROR):/d' \
   -e 's/\r$//' \
@@ -100,7 +100,7 @@  sed $E -e '/^Running target /,/^[[:space:]]+===.*Summary ===/!d' \
   >/tmp/o$$-$OBASE
 
 # Create a temporary file from the new file's interesting section.
-sed $E -e '/^Running target /,/^[[:space:]]+===.*Summary ===/!d' \
+sed $E -e "/$header/,/^[[:space:]]+===.*Summary ===/!d" \
   -e '/^[A-Z]+:/!d' \
   -e '/^(WARNING|ERROR):/d' \
   -e 's/\r$//' \