[Bug other/106216] New: new test case gcc.dg/tree-ssa/alias-access-path-13.c fails for 32 bits

[Bug other/106216] New: new test case gcc.dg/tree-ssa/alias-access-path-13.c fails for 32 bits

[seurer at gcc dot gnu.org]

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

            Bug ID: 106216
           Summary: new test case gcc.dg/tree-ssa/alias-access-path-13.c
                    fails for 32 bits
           Product: gcc
           Version: 13.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: other
          Assignee: unassigned at gcc dot gnu.org
          Reporter: seurer at gcc dot gnu.org
  Target Milestone: ---

g:b110e5283e368b5377e04766e4ff82cd52634208, r13-1460-gb110e5283e368b

This test is failing on 32 bit big endian powerpc64.

make  -k check-gcc RUNTESTFLAGS="--target_board=unix'{-m32}'
tree-ssa.exp=gcc.dg/tree-ssa/alias-access-path-13.c"
FAIL: gcc.dg/tree-ssa/alias-access-path-13.c scan-tree-dump-times fre1 "return
123" 1
# of expected passes            1
# of unexpected failures        1


commit b110e5283e368b5377e04766e4ff82cd52634208
Author: Martin Jambor <mjambor@suse.cz>
Date:   Fri Jul 1 20:57:18 2022 +0200

    tree-sra: Fix union handling in build_reconstructed_reference

[Bug other/106216] new test case gcc.dg/tree-ssa/alias-access-path-13.c fails for 32 bits

[jamborm at gcc dot gnu.org]

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

Martin Jambor <jamborm at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|                            |2022-07-06
           Assignee|unassigned at gcc dot gnu.org      |jamborm at gcc dot gnu.org
             Status|UNCONFIRMED                 |ASSIGNED
     Ever confirmed|0                           |1

[Bug testsuite/106216] new test case gcc.dg/tree-ssa/alias-access-path-13.c fails for 32 bits

[cvs-commit at gcc dot gnu.org]

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

--- Comment #1 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Martin Jambor <jamborm@gcc.gnu.org>:

https://gcc.gnu.org/g:b984b84cbe4bf026edef2ba37685f3958a1dc1cf

commit r13-1578-gb984b84cbe4bf026edef2ba37685f3958a1dc1cf
Author: Martin Jambor <mjambor@suse.cz>
Date:   Fri Jul 8 18:12:26 2022 +0200

    testsuite: Fix tree-ssa/alias-access-path-13.c on 32bit platforms (PR
106216)

    For gcc.dg/tree-ssa/alias-access-path-13.c to work, SRA must think of
    accesses to foo.inn.val and to foo itself as different ones, i.e. they
    need to have different offset and size, which on 32bit platforms they
    do not.  Fixed by replacing a dummy long int field of the union with a
    struct of two integers.

    Tested by:
      make -k check-gcc RUNTESTFLAGS="tree-ssa.exp=alias-access-path-13.c" and
      make -k check-gcc RUNTESTFLAGS="--target_board=unix'{-m32}'
tree-ssa.exp=alias-access-path-13.c"
    on an x86_64-linux, also with patched SRA to verify it still tests the
    original intent.

    gcc/testsuite/ChangeLog:

    2022-07-08  Martin Jambor  <mjambor@suse.cz>

            PR testsuite/106216
            * gcc.dg/tree-ssa/alias-access-path-13.c (union foo): Replace a
long
            int field with a struct that is larger than an int also on 32bit
            platforms.

[Bug testsuite/106216] new test case gcc.dg/tree-ssa/alias-access-path-13.c fails for 32 bits

[jamborm at gcc dot gnu.org]

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

Martin Jambor <jamborm at gcc dot gnu.org> changed:

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

--- Comment #2 from Martin Jambor <jamborm at gcc dot gnu.org> ---
Should work now.