diff mbox series

Fix unresolved tests I introduced into g++.dg/lto

Message ID 20190101131142.zy4fc5uwufi5oxh5@kam.mff.cuni.cz
State New
Headers show
Series Fix unresolved tests I introduced into g++.dg/lto | expand

Commit Message

Jan Hubicka Jan. 1, 2019, 1:11 p.m. UTC 
Hi,
this patch fixes the unresolved tests i accidentally introduced to 
lto directory.  I ended up droping the tree scan for now as I do not
know how to get that working and both testcases also tests runtime.

Honza

	* g++.dg/lto/devirt-13_0.C: Drop broken scan of ssa dump.
	* g++.dg/lto/devirt-14_0.C: Drop broken scan of ssa dump.
	* g++.dg/lto/devirt-23_0.C: Add -fdump-ipa-cp.

Comments

Jan Hubicka Jan. 1, 2019, 1:34 p.m. UTC | #1 
Hi,
this patch does housekeeping for ipa directory.
I still plan to port more of the non-LTO testcases into LTO variant (to
be sure LTO devirtualization works same way as non-LTO) but will do that
after handling bugs and other higher priority tasks.

Honza

	* g++.dg/ipa/devirt-36.C: Add dg-do-compile.
	* g++.dg/ipa/devirt-53.C: Fix scan template.
Index: g++.dg/ipa/devirt-36.C
===================================================================
--- g++.dg/ipa/devirt-36.C	(revision 267494)
+++ g++.dg/ipa/devirt-36.C	(working copy)
@@ -1,3 +1,4 @@
+/* { dg-do compile } */
 /* { dg-options "-O2 -fdump-ipa-devirt-details -fdump-tree-fre1-details"  } */
 struct A {virtual int t(void) {return 1;}};
 struct B:A {B(); virtual int t(void) {return 2;}};
Index: g++.dg/ipa/devirt-53.C
===================================================================
--- g++.dg/ipa/devirt-53.C	(revision 267494)
+++ g++.dg/ipa/devirt-53.C	(working copy)
@@ -55,4 +55,4 @@ void fn1() {
   q(new f::L).s(v);
 }
 /* Check that f::d appears as possible target.  */
-/* { dg-final { scan-tree-dump "f::d" "fre"  } } */
+/* { dg-final { scan-tree-dump "f::d" "fre1"  } } */
diff mbox series

Patch

Index: g++.dg/lto/devirt-13_0.C
===================================================================
--- g++.dg/lto/devirt-13_0.C	(revision 267494)
+++ g++.dg/lto/devirt-13_0.C	(working copy)
@@ -1,5 +1,5 @@ 
 /* { dg-lto-do run } */
 /* Call to foo should be devirtualized because there are no derived types of A.  */
-/* { dg-lto-options "-O2 -flto -fdump-tree-ssa"  } */
+/* { dg-lto-options "-O2 -flto"  } */
 #include "../ipa/devirt-13.C"
-/* { dg-final { scan-tree-dump-times "OBJ_TYPE_REF" 0 "ssa"} } */
+/* Ideally we should also { scan-tree-dump-times "OBJ_TYPE_REF" 0 "ssa"}.  */
Index: g++.dg/lto/devirt-14_0.C
===================================================================
--- g++.dg/lto/devirt-14_0.C	(revision 267494)
+++ g++.dg/lto/devirt-14_0.C	(working copy)
@@ -1,4 +1,4 @@ 
 /* { dg-lto-do run } */
 /* { dg-lto-options "-O2 -fdump-tree-ssa"  } */
 #include "../ipa/devirt-14.C"
-/* { dg-final { scan-tree-dump-not "A.*foo" "ssa"} } */
+/* Ideally we should also { scan-tree-dump-not "A.*foo" "ssa"} } */
Index: g++.dg/lto/devirt-23_0.C
===================================================================
--- g++.dg/lto/devirt-23_0.C	(revision 267494)
+++ g++.dg/lto/devirt-23_0.C	(working copy)
@@ -1,4 +1,4 @@ 
 /* { dg-lto-do run } */
-/* { dg-lto-options { "-O3 -fno-early-inlining -fno-ipa-sra -flto -fno-devirtualize-speculatively" } } */
+/* { dg-lto-options { "-O3 -fno-early-inlining -fno-ipa-sra -fdump-ipa-cp -flto -fno-devirtualize-speculatively" } } */
 #include "../ipa/devirt-23.C"
 /* { dg-final { scan-wpa-ipa-dump "Discovered a virtual call to" "cp" { xfail *-*-* } } } */