diff mbox series

[committed] testsuite: Fix up refactored scanltranstree.exp functions [PR116522]

Message ID ZtBPSCZiFEFQT47K@arm.com
State New
Headers show
Series [committed] testsuite: Fix up refactored scanltranstree.exp functions [PR116522] | expand

Commit Message

Alex Coplan Aug. 29, 2024, 10:36 a.m. UTC 
When adding RTL variants of the scan-ltrans-tree* functions in:
r15-3254-g3f51f0dc88ec21c1ec79df694200f10ef85915f4
I messed up the name of the underlying scan function to invoke.  The
code currently attempts to invoke functions named
scan{,-not,-dem,-dem-not} but should instead be invoking
scan-dump{,-not,-dem,-dem-not}.  This patch fixes that.

I missed this in testing because dg-cmp-results.sh (at least by default)
doesn't report new ERRORs.  I'll use a more robust way of comparing
test results in the future.

The problem didn't affect the scan-ltrans-{tree,rtl}-dump-times
functions, and I only spot-checked a test using one of those functions.
Apologies for the breakage.

Tested on aarch64.  No regressions, and verified that the ERROR in
gcc.dg/ipa/ipa-icf-38.c goes away.  Pushing to trunk as obvious.

Alex

gcc/testsuite/ChangeLog:

	PR testsuite/116522
	* lib/scanltranstree.exp: Fix name of underlying scan function
	used for scan-ltrans-{tree,rtl}-dump{,-not,-dem,-dem-not}.
diff mbox series

Patch

diff --git a/gcc/testsuite/lib/scanltranstree.exp b/gcc/testsuite/lib/scanltranstree.exp
index a7d4de3765f..3d85813ea2f 100644
--- a/gcc/testsuite/lib/scanltranstree.exp
+++ b/gcc/testsuite/lib/scanltranstree.exp
@@ -24,7 +24,7 @@  load_lib scandump.exp
 foreach ir { tree rtl } {
     foreach modifier { {} -not -dem -dem-not } {
 	eval [string map [list @NAME@ scan-ltrans-$ir-dump$modifier \
-			       @SCAN@ scan$modifier \
+			       @SCAN@ scan-dump$modifier \
 			       @TYPE@ ltrans-$ir \
 			       @SUFFIX@ [string index $ir 0]] {
 	    proc @NAME@ { args } {