Hi Martin,
> So the patch looks fine, only very very slightly binary is produced. I'm
> going to install the patch so that
> I can carry on more complex patches based on this one.
it seems you didn't properly test the testsuite part: I see
+UNRESOLVED: gcc.dg/tree-prof/update-loopch.c scan-tree-dump switchlower
"Removing basic block"
+UNRESOLVED: gcc.dg/tree-prof/update-loopch.c scan-tree-dump switchlower "loop
depth 1, count 33333"
+UNRESOLVED: gcc.dg/tree-prof/update-loopch.c scan-tree-dump-not switchlower
"Invalid sum"
+UNRESOLVED: gcc.dg/tree-prof/update-loopch.c scan-tree-dump-not switchlower
"loop depth 1, count 33332"
everywhere. The log has
gcc.dg/tree-prof/update-loopch.c: dump file does not exist
Obviously you forgot the adapt the dg-final* files for the dumpfile
name. If I do, three of the failures go away, but
FAIL: gcc.dg/tree-prof/update-loopch.c scan-tree-dump switchlower1 "Removing
basic block"
remains (on 32 and 64-bit Linux/x86_64).
Please fix.
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University