[Bug testsuite/87802] [9 regression] g++.dg/vect/slp-pr87105.cc fails starting with r265522
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87802 Richard Biener changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|--- |FIXED --- Comment #11 from Richard Biener --- Fixed.
[Bug testsuite/87802] [9 regression] g++.dg/vect/slp-pr87105.cc fails starting with r265522
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87802 --- Comment #10 from Richard Biener --- Author: rguenth Date: Wed Oct 31 08:48:59 2018 New Revision: 265655 URL: https://gcc.gnu.org/viewcvs?rev=265655=gcc=rev Log: 2018-10-31 Richard Biener PR testsuite/87802 * g++.dg/vect/slp-pr87105.cc: XFAIL for strict alignment targets. Modified: trunk/gcc/testsuite/ChangeLog trunk/gcc/testsuite/g++.dg/vect/slp-pr87105.cc
[Bug testsuite/87802] [9 regression] g++.dg/vect/slp-pr87105.cc fails starting with r265522
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87802 --- Comment #9 from seurer at gcc dot gnu.org --- Yes, it works with that patch.
[Bug testsuite/87802] [9 regression] g++.dg/vect/slp-pr87105.cc fails starting with r265522
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87802
--- Comment #8 from Richard Biener ---
Ah, ok.
/home/seurer/gcc/gcc-test/gcc/testsuite/g++.dg/vect/slp-pr87105.cc:98:1:
missed: not vectorized: bad data alignment in basic block.
looks like with the flags we do not support unaligned accesses. Removing
the -mno-allow-movmisalign flag fixes this.
Does the following work? If so please go ahead and install it.
Index: gcc/testsuite/g++.dg/vect/slp-pr87105.cc
===
--- gcc/testsuite/g++.dg/vect/slp-pr87105.cc(revision 265619)
+++ gcc/testsuite/g++.dg/vect/slp-pr87105.cc(working copy)
@@ -99,7 +99,7 @@ void quadBoundingBoxA(const Point bez[3]
// We should have if-converted everything down to straight-line code
// { dg-final { scan-tree-dump-times "" 1 "slp2" } }
-// { dg-final { scan-tree-dump-times "basic block part vectorized" 1 "slp2" }
}
+// { dg-final { scan-tree-dump-times "basic block part vectorized" 1 "slp2" {
xfail { { ! vect_element_align } && { ! vect_hw_misalign } } } } }
// It's a bit awkward to detect that all stores were vectorized but the
// following more or less does the trick
-// { dg-final { scan-tree-dump "vect_iftmp\[^\r\m\]* = MIN" "slp2" } }
+// { dg-final { scan-tree-dump "vect_iftmp\[^\r\m\]* = MIN" "slp2" { xfail { {
! vect_element_align } && { ! vect_hw_misalign } } } } }
[Bug testsuite/87802] [9 regression] g++.dg/vect/slp-pr87105.cc fails starting with r265522
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87802 --- Comment #7 from seurer at gcc dot gnu.org --- Various output files attached and here's a run with -v so you can see all the options: seurer@granola:~/gcc/build/gcc-test$ /home/seurer/gcc/build/gcc-test/gcc/testsuite/g++/../../xg++ -v -B/home/seurer/gcc/build/gcc-test/gcc/testsuite/g++/../../ /home/seurer/gcc/gcc-test/gcc/testsuite/g++.dg/vect/slp-pr87105.cc -fno-diagnostics-show-caret -fno-diagnostics-show-line-numbers -fdiagnostics-color=never -nostdinc++ -I/home/seurer/gcc/build/gcc-test/powerpc64-unknown-linux-gnu/libstdc++-v3/include/powerpc64-unknown-linux-gnu -I/home/seurer/gcc/build/gcc-test/powerpc64-unknown-linux-gnu/libstdc++-v3/include -I/home/seurer/gcc/gcc-test/libstdc++-v3/libsupc++ -I/home/seurer/gcc/gcc-test/libstdc++-v3/include/backward -I/home/seurer/gcc/gcc-test/libstdc++-v3/testsuite/util -fmessage-length=0 -std=c++14 -O2 -ftree-vectorize -fno-vect-cost-model -maltivec -mvsx -mno-allow-movmisalign -fdump-tree-slp-details -ffast-math -E -o ./slp-pr8105.i Reading specs from /home/seurer/gcc/build/gcc-test/gcc/testsuite/g++/../../specs COLLECT_GCC=/home/seurer/gcc/build/gcc-test/gcc/testsuite/g++/../../xg++ Target: powerpc64-unknown-linux-gnu Configured with: /home/seurer/gcc/gcc-test/configure --prefix=/home/seurer/gcc/install/gcc-test --enable-languages=c,fortran,c++ --with-cpu=power7 --disable-bootstrap --disable-multilib --with-as=/home/seurer/binutils/install/binutils-2.26/bin/as --with-ld=/home/seurer/binutils/install/binutils-2.26/bin/ld Thread model: posix gcc version 9.0.0 20181026 (experimental) [trunk revision 265522] (GCC) COLLECT_GCC_OPTIONS='-fdiagnostics-color=never' '-v' '-B' '/home/seurer/gcc/build/gcc-test/gcc/testsuite/g++/../../' '-fno-diagnostics-show-caret' '-fno-diagnostics-show-line-numbers' '-nostdinc++' '-I' '/home/seurer/gcc/build/gcc-test/powerpc64-unknown-linux-gnu/libstdc++-v3/include/powerpc64-unknown-linux-gnu' '-I' '/home/seurer/gcc/build/gcc-test/powerpc64-unknown-linux-gnu/libstdc++-v3/include' '-I' '/home/seurer/gcc/gcc-test/libstdc++-v3/libsupc++' '-I' '/home/seurer/gcc/gcc-test/libstdc++-v3/include/backward' '-I' '/home/seurer/gcc/gcc-test/libstdc++-v3/testsuite/util' '-fmessage-length=0' '-std=c++14' '-O2' '-ftree-vectorize' '-fvect-cost-model=unlimited' '-maltivec' '-mvsx' '-mno-allow-movmisalign' '-fdump-tree-slp-details' '-ffast-math' '-E' '-o' './slp-pr8105.i' '-shared-libgcc' '-mcpu=power7' /home/seurer/gcc/build/gcc-test/gcc/testsuite/g++/../../cc1plus -E -quiet -nostdinc++ -v -I /home/seurer/gcc/build/gcc-test/powerpc64-unknown-linux-gnu/libstdc++-v3/include/powerpc64-unknown-linux-gnu -I /home/seurer/gcc/build/gcc-test/powerpc64-unknown-linux-gnu/libstdc++-v3/include -I /home/seurer/gcc/gcc-test/libstdc++-v3/libsupc++ -I /home/seurer/gcc/gcc-test/libstdc++-v3/include/backward -I /home/seurer/gcc/gcc-test/libstdc++-v3/testsuite/util -iprefix /home/seurer/gcc/build/gcc-test/gcc/../lib/gcc/powerpc64-unknown-linux-gnu/9.0.0/ -isystem /home/seurer/gcc/build/gcc-test/gcc/testsuite/g++/../../include -isystem /home/seurer/gcc/build/gcc-test/gcc/testsuite/g++/../../include-fixed -D_GNU_SOURCE -D__unix__ -D__gnu_linux__ -D__linux__ -Asystem=linux -Asystem=unix -Asystem=posix /home/seurer/gcc/gcc-test/gcc/testsuite/g++.dg/vect/slp-pr87105.cc -o ./slp-pr8105.i -maltivec -mvsx -mno-allow-movmisalign -mcpu=power7 -std=c++14 -fdiagnostics-color=never -fno-diagnostics-show-caret -fno-diagnostics-show-line-numbers -fmessage-length=0 -ftree-vectorize -fvect-cost-model=unlimited -fdump-tree-slp-details -ffast-math -O2 ignoring nonexistent directory "/home/seurer/gcc/build/gcc-test/gcc/../lib/gcc/powerpc64-unknown-linux-gnu/9.0.0/include" ignoring nonexistent directory "/home/seurer/gcc/build/gcc-test/gcc/../lib/gcc/powerpc64-unknown-linux-gnu/9.0.0/include-fixed" ignoring nonexistent directory "/home/seurer/gcc/build/gcc-test/gcc/../lib/gcc/powerpc64-unknown-linux-gnu/9.0.0/../../../../powerpc64-unknown-linux-gnu/include" ignoring nonexistent directory "/home/seurer/gcc/build/gcc-test/gcc/../lib/gcc/../../lib/gcc/powerpc64-unknown-linux-gnu/9.0.0/include" ignoring nonexistent directory "/home/seurer/gcc/build/gcc-test/gcc/../lib/gcc/../../include" ignoring nonexistent directory "/home/seurer/gcc/build/gcc-test/gcc/../lib/gcc/../../lib/gcc/powerpc64-unknown-linux-gnu/9.0.0/include-fixed" ignoring nonexistent directory "/home/seurer/gcc/build/gcc-test/gcc/../lib/gcc/../../lib/gcc/powerpc64-unknown-linux-gnu/9.0.0/../../../../powerpc64-unknown-linux-gnu/include" #include "..." search starts here: #include <...> search starts here: /home/seurer/gcc/build/gcc-test/powerpc64-unknown-linux-gnu/libstdc++-v3/include/powerpc64-unknown-linux-gnu /home/seurer/gcc/build/gcc-test/powerpc64-unknown-linux-gnu/libstdc++-v3/include /home/seurer/gcc/gcc-test/libstdc++-v3/libsupc++ /home/seurer/gcc/gcc-test/libstdc++-v3/include/backward /home/seurer/gcc/gcc-test/libstdc++-v3/testsuite/util
[Bug testsuite/87802] [9 regression] g++.dg/vect/slp-pr87105.cc fails starting with r265522
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87802 --- Comment #6 from seurer at gcc dot gnu.org --- Created attachment 44931 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=44931=edit preprocessed source
[Bug testsuite/87802] [9 regression] g++.dg/vect/slp-pr87105.cc fails starting with r265522
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87802 --- Comment #5 from seurer at gcc dot gnu.org --- Created attachment 44930 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=44930=edit intermediate text
[Bug testsuite/87802] [9 regression] g++.dg/vect/slp-pr87105.cc fails starting with r265522
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87802 --- Comment #4 from seurer at gcc dot gnu.org --- Created attachment 44929 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=44929=edit assembler
[Bug testsuite/87802] [9 regression] g++.dg/vect/slp-pr87105.cc fails starting with r265522
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87802 Richard Biener changed: What|Removed |Added Component|other |testsuite --- Comment #3 from Richard Biener --- Testsuite issue for sure.
