On Wed, Oct 10, 2012 at 09:48:02AM +0000, Kumar, Venkataramanan wrote:
> +2012-10-09  Venkataramanan Kumar  <venkataramanan.ku...@amd.com>
> +
> +     PR testsuite/53397
> +     * gcc/testsuite/gcc.dg: Remove test cases pr53397-1.c and pr53397-2.c
> +     they are moved to gcc.target/i386 as they are specific to i386/x86_64.
> +     * gcc/testsuite/gcc.target/i386/pr53397-1.c: New test 
> +     Checks we are prefetching for loop invariant steps
> +     * gcc/testsuite/gcc.target/i386/pr53397-2.c: New test
> +     Checks we are not prefetching for loop variant steps     

The ChangeLog entry is wrong.  No gcc/testsuite/ paths, sentences should
end with dot, no need to explain why you are doing the change.
Just write:
        PR testsuite/53397
        * gcc.dg/pr53397-1.c: Moved to...
        * gcc.target/i386/pr53397-1.c: ... here.  Add -msse2 to dg-options.
        * gcc.dg/pr53397-2.c: Moved to...
        * gcc.target/i386/pr53397-2.c: ... here.  Add -msse2 to dg-options.

Ok with that change (though, preferrably just do svn mv of the files, then
add -msse2, edit ChangeLog and commit).

        Jakub

Reply via email to