OK with the change below.
Diego. http://codereview.appspot.com/4744043/diff/3001/gcc/testsuite/lib/dg-pph.exp File gcc/testsuite/lib/dg-pph.exp (right): http://codereview.appspot.com/4744043/diff/3001/gcc/testsuite/lib/dg-pph.exp#newcode131 gcc/testsuite/lib/dg-pph.exp:131: set adiff [catch {exec diff "$bname.s-pph" "$bname.s+pph"} diff_result] 131 set adiff [catch {exec diff "$bname.s-pph" "$bname.s+pph"} diff_result] You don't need this, if instead of summing the diff you sum $bname.s+pph. The logic would then be: if there is a difference between $bname.s-pph and $bname.s+pph, we checksum $bname.s+pph. If the new checksum is different than the stored one, it means that $bname.s+pph is different from $bname.s-pph in a different way. This has the benefit of: - Being slightly faster. - Simplifies the generation of checksums. We no longer need to checksum the difference between the two .s files, we just need to checksum the .s+pph file http://codereview.appspot.com/4744043/
