[Bug middle-end/80815] wrong code because of broken runtime alias check in vectorizer

2017-08-07 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80815 amker at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|--

[Bug middle-end/80815] wrong code because of broken runtime alias check in vectorizer

2017-08-02 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80815 --- Comment #8 from amker at gcc dot gnu.org --- vect_perm added for the the test case. It should be bypassed now on sparc-sun-solaris2.12?

[Bug middle-end/80815] wrong code because of broken runtime alias check in vectorizer

2017-06-02 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80815 --- Comment #7 from amker at gcc dot gnu.org --- (In reply to Rainer Orth from comment #6) > Created attachment 41456 [details] > sparc-sun-solaris2.12 pr80815-3.c.156t.vect Thanks for reporting, I will investigate it and disable on targets if it

[Bug middle-end/80815] wrong code because of broken runtime alias check in vectorizer

2017-06-02 Thread ro at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80815 Rainer Orth changed: What|Removed |Added CC||ro at gcc dot gnu.org --- Comment #6 from

[Bug middle-end/80815] wrong code because of broken runtime alias check in vectorizer

2017-05-26 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80815 --- Comment #5 from amker at gcc dot gnu.org --- Author: amker Date: Fri May 26 14:18:26 2017 New Revision: 248512 URL: https://gcc.gnu.org/viewcvs?rev=248512&root=gcc&view=rev Log: PR tree-optimization/80815 * tree-data-ref.c (pr

[Bug middle-end/80815] wrong code because of broken runtime alias check in vectorizer

2017-05-19 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80815 --- Comment #4 from amker at gcc dot gnu.org --- Also the three cases: /* If the left segment does not extend beyond the start of the right segment the new segment length is that of the right plus the segment di

[Bug middle-end/80815] wrong code because of broken runtime alias check in vectorizer

2017-05-19 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80815 --- Comment #3 from amker at gcc dot gnu.org --- (In reply to rguent...@suse.de from comment #2) > On Fri, 19 May 2017, amker at gcc dot gnu.org wrote: > > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80815 > > > > --- Comment #1 from amker at

[Bug middle-end/80815] wrong code because of broken runtime alias check in vectorizer

2017-05-19 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80815 --- Comment #2 from rguenther at suse dot de --- On Fri, 19 May 2017, amker at gcc dot gnu.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80815 > > --- Comment #1 from amker at gcc dot gnu.org --- > GCC uses vect_factor as minimal se

[Bug middle-end/80815] wrong code because of broken runtime alias check in vectorizer

2017-05-19 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80815 --- Comment #1 from amker at gcc dot gnu.org --- GCC uses vect_factor as minimal segment length for dr_b when merging alias pairs, I think it could be relaxed to vect_factor * abs (DR_STEP (dr_b)). Below test shows this change can merge additiona

[Bug middle-end/80815] wrong code because of broken runtime alias check in vectorizer

2017-05-18 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80815 amker at gcc dot gnu.org changed: What|Removed |Added Keywords||wrong-code Priority|