public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug middle-end/102516] New: [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893
@ 2021-09-28 15:00 clyon at gcc dot gnu.org
  2021-09-29  6:13 ` [Bug tree-optimization/102516] " rguenth at gcc dot gnu.org
                   ` (8 more replies)
  0 siblings, 9 replies; 10+ messages in thread
From: clyon at gcc dot gnu.org @ 2021-09-28 15:00 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102516

            Bug ID: 102516
           Summary: [12 regression] pr65947-13.c and vect-alias-check-18.c
                    fail on armeb since r12-3893
           Product: gcc
           Version: 12.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
          Assignee: unassigned at gcc dot gnu.org
          Reporter: clyon at gcc dot gnu.org
  Target Milestone: ---

Since r12-3893 (g:6390c5047adb75960f86d56582e6322aaa4d9281), I've noticed
regressions on armeb targets:


gcc.dg/vect/pr65947-13.c: pattern found 1 times
FAIL: gcc.dg/vect/pr65947-13.c scan-tree-dump-times vect "LOOP VECTORIZED" 2
gcc.dg/vect/pr65947-13.c -flto -ffat-lto-objects : pattern found 1 times
FAIL: gcc.dg/vect/pr65947-13.c -flto -ffat-lto-objects  scan-tree-dump-times
vect "LOOP VECTORIZED" 2

FAIL: gcc:gcc.dg/vect/vect.exp=gcc.dg/vect/vect-alias-check-18.c -flto
-ffat-lto-objects  scan-tree-dump vect "using an index-based WAR/WAW test"
FAIL: gcc:gcc.dg/vect/vect.exp=gcc.dg/vect/vect-alias-check-18.c scan-tree-dump
vect "using an index-based WAR/WAW test"

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug tree-optimization/102516] [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893
  2021-09-28 15:00 [Bug middle-end/102516] New: [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893 clyon at gcc dot gnu.org
@ 2021-09-29  6:13 ` rguenth at gcc dot gnu.org
  2021-09-29 12:44 ` rguenth at gcc dot gnu.org
                   ` (7 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2021-09-29  6:13 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102516

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |12.0
   Last reconfirmed|                            |2021-09-29
     Ever confirmed|0                           |1
          Component|middle-end                  |tree-optimization
             Status|UNCONFIRMED                 |ASSIGNED

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
I will have a look.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug tree-optimization/102516] [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893
  2021-09-28 15:00 [Bug middle-end/102516] New: [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893 clyon at gcc dot gnu.org
  2021-09-29  6:13 ` [Bug tree-optimization/102516] " rguenth at gcc dot gnu.org
@ 2021-09-29 12:44 ` rguenth at gcc dot gnu.org
  2021-09-29 13:12 ` clyon at gcc dot gnu.org
                   ` (6 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2021-09-29 12:44 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102516

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |WAITING

--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
How do I need to configure GCC?  --target=armeb-linux-gnu isn't supported.

Note this looks like peeling related, maybe it's also fixed after
g:4c7731081647c22cbd249dc0faa20c3df9ed6411

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug tree-optimization/102516] [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893
  2021-09-28 15:00 [Bug middle-end/102516] New: [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893 clyon at gcc dot gnu.org
  2021-09-29  6:13 ` [Bug tree-optimization/102516] " rguenth at gcc dot gnu.org
  2021-09-29 12:44 ` rguenth at gcc dot gnu.org
@ 2021-09-29 13:12 ` clyon at gcc dot gnu.org
  2022-04-12 15:31 ` rearnsha at gcc dot gnu.org
                   ` (5 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: clyon at gcc dot gnu.org @ 2021-09-29 13:12 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102516

--- Comment #3 from Christophe Lyon <clyon at gcc dot gnu.org> ---
The target name is armeb-linux-gnueabi or armeb-linux-gnueabihf

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug tree-optimization/102516] [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893
  2021-09-28 15:00 [Bug middle-end/102516] New: [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893 clyon at gcc dot gnu.org
                   ` (2 preceding siblings ...)
  2021-09-29 13:12 ` clyon at gcc dot gnu.org
@ 2022-04-12 15:31 ` rearnsha at gcc dot gnu.org
  2022-04-12 21:11 ` clyon at gcc dot gnu.org
                   ` (4 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: rearnsha at gcc dot gnu.org @ 2022-04-12 15:31 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102516

Richard Earnshaw <rearnsha at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |NEW

--- Comment #4 from Richard Earnshaw <rearnsha at gcc dot gnu.org> ---
I can reproduce this on an normal arm-eabi cross compiler with

$build/gcc/xgcc -B$build/gcc/ $src/gcc/testsuite/gcc.dg/vect/pr65947-13.c
-mbig-endian -march=armv7-a+simd -fdiagnostics-plain-output -mfloat-abi=softfp
-ffast-math -ftree-vectorize -fno-tree-loop-distribute-patterns
-fno-vect-cost-model -fno-common -O2 -fdump-tree-vect-details --param
vect-epilogues-nomask=0 -S -isystem
$build/arm-eabi/thumb/v7-a/nofp/newlib/targ-include -isystem
$src/newlib/libc/include -o pr65947-13.s

The test expects "LOOP VECTORIZED" to appear twice in the dump file, but it
only appears once.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug tree-optimization/102516] [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893
  2021-09-28 15:00 [Bug middle-end/102516] New: [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893 clyon at gcc dot gnu.org
                   ` (3 preceding siblings ...)
  2022-04-12 15:31 ` rearnsha at gcc dot gnu.org
@ 2022-04-12 21:11 ` clyon at gcc dot gnu.org
  2022-05-06  8:31 ` [Bug tree-optimization/102516] [12/13 " jakub at gcc dot gnu.org
                   ` (3 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: clyon at gcc dot gnu.org @ 2022-04-12 21:11 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102516

--- Comment #5 from Christophe Lyon <clyon at gcc dot gnu.org> ---
Indeed. I know that vectorization does not work on armeb as well as on arm
(little-endian), I was just wondering whether this regression was "expected"

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug tree-optimization/102516] [12/13 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893
  2021-09-28 15:00 [Bug middle-end/102516] New: [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893 clyon at gcc dot gnu.org
                   ` (4 preceding siblings ...)
  2022-04-12 21:11 ` clyon at gcc dot gnu.org
@ 2022-05-06  8:31 ` jakub at gcc dot gnu.org
  2022-10-19  9:34 ` rguenth at gcc dot gnu.org
                   ` (2 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: jakub at gcc dot gnu.org @ 2022-05-06  8:31 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102516

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|12.0                        |12.2

--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 12.1 is being released, retargeting bugs to GCC 12.2.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug tree-optimization/102516] [12/13 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893
  2021-09-28 15:00 [Bug middle-end/102516] New: [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893 clyon at gcc dot gnu.org
                   ` (5 preceding siblings ...)
  2022-05-06  8:31 ` [Bug tree-optimization/102516] [12/13 " jakub at gcc dot gnu.org
@ 2022-10-19  9:34 ` rguenth at gcc dot gnu.org
  2022-10-24 12:56 ` clyon at gcc dot gnu.org
  2023-02-21 15:04 ` rguenth at gcc dot gnu.org
  8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2022-10-19  9:34 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102516

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |WAITING

--- Comment #8 from Richard Biener <rguenth at gcc dot gnu.org> ---
Note the change wasn't expected.

Btw, with what Richard says in comment#4 I see LOOP VECTORIZED twice on recent
trunk.

So is this still an issue?  Can somebody bisect for the fix?

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug tree-optimization/102516] [12/13 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893
  2021-09-28 15:00 [Bug middle-end/102516] New: [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893 clyon at gcc dot gnu.org
                   ` (6 preceding siblings ...)
  2022-10-19  9:34 ` rguenth at gcc dot gnu.org
@ 2022-10-24 12:56 ` clyon at gcc dot gnu.org
  2023-02-21 15:04 ` rguenth at gcc dot gnu.org
  8 siblings, 0 replies; 10+ messages in thread
From: clyon at gcc dot gnu.org @ 2022-10-24 12:56 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102516

--- Comment #9 from Christophe Lyon <clyon at gcc dot gnu.org> ---
Indeed it works again on trunk, I bisected and it started with one of our
commits: r12-3958-g4c7731081647c22cbd249dc0faa20c3df9ed6411
Author: Richard Biener <rguenther@suse.de>
Date:   Wed Sep 29 11:18:23 2021 +0200

    Fix peeling for alignment with negative step

    The following fixes a regression causing us to no longer peel
    negative step loops for alignment.  With dr_misalignment now
    applying the bias for negative step we have to do the reverse
    when adjusting the misalignment for peeled DRs.

    2021-09-29  Richard Biener  <rguenther@suse.de>

            * tree-vect-data-refs.c (vect_dr_misalign_for_aligned_access):
            New helper.
            (vect_update_misalignment_for_peel): Use it to update
            misaligned to the value necessary for an aligned access.
            (vect_get_peeling_costs_all_drs): Likewise.
            (vect_enhance_data_refs_alignment): Likewise.

            * gcc.target/i386/vect-alignment-peeling-1.c: New testcase.
            * gcc.target/i386/vect-alignment-peeling-2.c: Likewise.


We still have
FAIL: gcc.dg/vect/pr65947-13.c scan-tree-dump-times vect "condition expression
based on integer induction." 2
but based on the original report, this was already present

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug tree-optimization/102516] [12/13 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893
  2021-09-28 15:00 [Bug middle-end/102516] New: [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893 clyon at gcc dot gnu.org
                   ` (7 preceding siblings ...)
  2022-10-24 12:56 ` clyon at gcc dot gnu.org
@ 2023-02-21 15:04 ` rguenth at gcc dot gnu.org
  8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2023-02-21 15:04 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102516

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |RESOLVED
         Resolution|---                         |FIXED

--- Comment #10 from Richard Biener <rguenth at gcc dot gnu.org> ---
Thus fixed.

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2023-02-21 15:04 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-28 15:00 [Bug middle-end/102516] New: [12 regression] pr65947-13.c and vect-alias-check-18.c fail on armeb since r12-3893 clyon at gcc dot gnu.org
2021-09-29  6:13 ` [Bug tree-optimization/102516] " rguenth at gcc dot gnu.org
2021-09-29 12:44 ` rguenth at gcc dot gnu.org
2021-09-29 13:12 ` clyon at gcc dot gnu.org
2022-04-12 15:31 ` rearnsha at gcc dot gnu.org
2022-04-12 21:11 ` clyon at gcc dot gnu.org
2022-05-06  8:31 ` [Bug tree-optimization/102516] [12/13 " jakub at gcc dot gnu.org
2022-10-19  9:34 ` rguenth at gcc dot gnu.org
2022-10-24 12:56 ` clyon at gcc dot gnu.org
2023-02-21 15:04 ` rguenth at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).