public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug middle-end/102808] New: [12 regression] sve/mask_gather_load_1.c fails since g:9b2ad21ab3ebc21a3408108327fa1a7cbedaf217
@ 2021-10-18  7:33 clyon at gcc dot gnu.org
  2021-10-18  8:45 ` [Bug middle-end/102808] " rguenth at gcc dot gnu.org
  2021-12-03 16:43 ` rsandifo at gcc dot gnu.org
  0 siblings, 2 replies; 3+ messages in thread
From: clyon at gcc dot gnu.org @ 2021-10-18  7:33 UTC (permalink / raw)
  To: gcc-bugs

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

            Bug ID: 102808
           Summary: [12 regression] sve/mask_gather_load_1.c fails since
                    g:9b2ad21ab3ebc21a3408108327fa1a7cbedaf217
           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: ---

Hi,

Since g:9b2ad21ab3ebc21a3408108327fa1a7cbedaf217, I have noticed a few
regressions in the aarch64 tests:

FAIL:
gcc:gcc.target/aarch64/sve/aarch64-sve.exp=gcc.target/aarch64/sve/mask_gather_load_1.c
scan-assembler-times \\tld1d\\tz[0-9]+\\.d, p[0-7]/z, \\[x[0-9]+, z[0-9]+\\.d,
lsl 3\\]\\n 9
FAIL:
gcc:gcc.target/aarch64/sve/aarch64-sve.exp=gcc.target/aarch64/sve/mask_gather_load_2.c
scan-assembler-times \\tld1d\\tz[0-9]+\\.d, p[0-7]/z, \\[x[0-9]+, z[0-9]+\\.d,
lsl 3\\]\\n 9
FAIL:
gcc:gcc.target/aarch64/sve/aarch64-sve.exp=gcc.target/aarch64/sve/mask_gather_load_3.c
scan-assembler-times \\tld1d\\tz[0-9]+\\.d, p[0-7]/z, \\[x[0-9]+,
z[0-9]+\\.d\\]\\n 9
FAIL:
gcc:gcc.target/aarch64/sve/aarch64-sve.exp=gcc.target/aarch64/sve/mask_gather_load_4.c
scan-assembler-times \\tld1d\\tz[0-9]+\\.d, p[0-7]/z, \\[x[0-9]+,
z[0-9]+\\.d\\]\\n 9
FAIL:
gcc:gcc.target/aarch64/sve/aarch64-sve.exp=gcc.target/aarch64/sve/mask_gather_load_7.c
scan-assembler-times \\tld1d\\tz[0-9]+\\.d, p[0-7]/z, \\[x[0-9]+, z[0-9]+\\.d,
lsl 3\\]\\n 36
FAIL:
gcc:gcc.target/aarch64/sve/aarch64-sve.exp=gcc.target/aarch64/sve/mask_scatter_store_1.c
scan-assembler-times \\tst1d\\tz[0-9]+\\.d, p[0-7], \\[x[0-9]+, z[0-9]+\\.d,
lsl 3\\]\\n 9
FAIL:
gcc:gcc.target/aarch64/sve/aarch64-sve.exp=gcc.target/aarch64/sve/mask_scatter_store_2.c
scan-assembler-times \\tst1d\\tz[0-9]+\\.d, p[0-7], \\[x[0-9]+, z[0-9]+\\.d,
lsl 3\\]\\n 9

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

* [Bug middle-end/102808] [12 regression] sve/mask_gather_load_1.c fails since g:9b2ad21ab3ebc21a3408108327fa1a7cbedaf217
  2021-10-18  7:33 [Bug middle-end/102808] New: [12 regression] sve/mask_gather_load_1.c fails since g:9b2ad21ab3ebc21a3408108327fa1a7cbedaf217 clyon at gcc dot gnu.org
@ 2021-10-18  8:45 ` rguenth at gcc dot gnu.org
  2021-12-03 16:43 ` rsandifo at gcc dot gnu.org
  1 sibling, 0 replies; 3+ messages in thread
From: rguenth at gcc dot gnu.org @ 2021-10-18  8:45 UTC (permalink / raw)
  To: gcc-bugs

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |12.0
                 CC|                            |rguenth at gcc dot gnu.org,
                   |                            |rsandifo at gcc dot gnu.org
           Keywords|                            |missed-optimization

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
Can arm folks please have a look here?  Cases like

    for (int i = 0; i < n; ++i)                                         \
      if (cmp1[i] == cmp2[i])                                           \
        dest[i] += src[indices[i]];

involve undefined integer overflow ops conditionally executed which are now
rewritten to unsigned.  On x86 the only fallout in gather/scatter testcases
was with invariants and vect_check_gather_scatter plus/minus handling but
I mitigated that.

Note the testcases only scan for assembler, not for vectorization results
and thus they might simply be too fragile.

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

* [Bug middle-end/102808] [12 regression] sve/mask_gather_load_1.c fails since g:9b2ad21ab3ebc21a3408108327fa1a7cbedaf217
  2021-10-18  7:33 [Bug middle-end/102808] New: [12 regression] sve/mask_gather_load_1.c fails since g:9b2ad21ab3ebc21a3408108327fa1a7cbedaf217 clyon at gcc dot gnu.org
  2021-10-18  8:45 ` [Bug middle-end/102808] " rguenth at gcc dot gnu.org
@ 2021-12-03 16:43 ` rsandifo at gcc dot gnu.org
  1 sibling, 0 replies; 3+ messages in thread
From: rsandifo at gcc dot gnu.org @ 2021-12-03 16:43 UTC (permalink / raw)
  To: gcc-bugs

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

rsandifo at gcc dot gnu.org <rsandifo at gcc dot gnu.org> changed:

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

--- Comment #2 from rsandifo at gcc dot gnu.org <rsandifo at gcc dot gnu.org> ---
Fixed by g:8bb81b3c4945dbae8d490e7a4054ddeb5a3b506c.
(Forgot there was a PR for this, oops.)

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

end of thread, other threads:[~2021-12-03 16:43 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-18  7:33 [Bug middle-end/102808] New: [12 regression] sve/mask_gather_load_1.c fails since g:9b2ad21ab3ebc21a3408108327fa1a7cbedaf217 clyon at gcc dot gnu.org
2021-10-18  8:45 ` [Bug middle-end/102808] " rguenth at gcc dot gnu.org
2021-12-03 16:43 ` rsandifo 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).