[PR 83178] Adjust scan-dump-times count of devirt-22.C

Martin Jambor mjambor@suse.cz
Thu Jan 11 13:54:00 GMT 2018


Hi,

IPA-CP does an extra devirtualization in devirt-22.C now because Honza's
profiling changes have caused an extra edge to be maybe_hot_p.  The scan
dump is guarded with scan-dump-times, I assume so that we are alerted to
any changes in behavior of the patch, like now.

I have looked at the behavior of IPA-CP and also reviewed the new
profile info, which seems better or at least not worse than before (more
info in my comment to the bug) and therefore I would like to propose to
adjust the test dump scan expected count in the patch below.

OK for trunk?

Thanks,

Martin


2018-01-11  Martin Jambor  <mjambor@suse.cz>

	PR ipa/83178
	* g++.dg/ipa/devirt-22.C: Adjust scan-dump-times count.


diff --git a/gcc/testsuite/g++.dg/ipa/devirt-22.C b/gcc/testsuite/g++.dg/ipa/devirt-22.C
index 0201f34eade..7ec4f436145 100644
--- a/gcc/testsuite/g++.dg/ipa/devirt-22.C
+++ b/gcc/testsuite/g++.dg/ipa/devirt-22.C
@@ -37,4 +37,4 @@ void update_sources() {
 /* Note that we miss one devirtualization because we are not able to track the
    vtbl store in destructor.  
    Previously we devirtualized to C::m_fn1 instead of B::m_fn1.  */
-/* { dg-final { scan-ipa-dump-times "Discovered a virtual call to a known target" 1 "cp"  } } */
+/* { dg-final { scan-ipa-dump-times "Discovered a virtual call to a known target" 2 "cp"  } } */



More information about the Gcc-patches mailing list