[Bug ipa/64192] New: [5 Regression] r218369 causes some regressions with -m32.

[Bug ipa/64192] New: [5 Regression] r218369 causes some regressions with -m32.

[dominiq at lps dot ens.fr]

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

            Bug ID: 64192
           Summary: [5 Regression] r218369 causes some regressions with
                    -m32.
           Product: gcc
           Version: 5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: ipa
          Assignee: unassigned at gcc dot gnu.org
          Reporter: dominiq at lps dot ens.fr

[Bug ipa/64192] [5 Regression] r218369 causes some regressions with -m32.

[dominiq at lps dot ens.fr]

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

Dominique d'Humieres <dominiq at lps dot ens.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2014-12-05
                 CC|                            |jamborm at gcc dot gnu.org
     Ever confirmed|0                           |1

--- Comment #1 from Dominique d'Humieres <dominiq at lps dot ens.fr> ---
Hit return too soon!-(

Revision r218369 causes some regressions with -m32

FAIL: c-c++-common/ubsan/object-size-9.c   -O2 -flto -fuse-linker-plugin
-fno-fat-lto-objects  output pattern test, is
c-c++-common/ubsan/object-size-9.c:21:11: runtime error: load of address
0x7fff54fb3cdc with insufficient space for an object of type 'char'
FAIL: gcc.c-torture/execute/pr37573.c   -O3 -fomit-frame-pointer  execution
test
FAIL: gcc.c-torture/execute/pr37573.c   -O3 -fomit-frame-pointer
-funroll-all-loops -finline-functions  execution test
FAIL: gcc.c-torture/execute/pr37573.c   -O3 -fomit-frame-pointer -funroll-loops
 execution test
FAIL: gcc.c-torture/execute/pr37573.c   -O3 -g  execution test
FAIL: gcc.dg/vect/pr60196-1.c execution test
FAIL: gcc.dg/vect/pr60196-1.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-multitypes-11.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-multitypes-12.c -flto -ffat-lto-objects execution test

see for instance https://gcc.gnu.org/ml/gcc-regression/2014-12/msg00136.html.

[Bug ipa/64192] [5 Regression] r218369 causes some regressions with -m32.

[jamborm at gcc dot gnu.org]

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

--- Comment #2 from Martin Jambor <jamborm at gcc dot gnu.org> ---
Hm, the only one I can reproduce is the gcc.dg/vect/pr60196-1.c
failure, but I will start with that and have a look.  Hopefully it is
the same issue.

[Bug ipa/64192] [5 Regression] r218369 causes some regressions with -m32.

[schwab@linux-m68k.org]

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

--- Comment #3 from Andreas Schwab <schwab@linux-m68k.org> ---
This is also causing unaligned accesses in the compiler on ia64, for example in
function ipa-inline-analysis.c:account_size_time:

      new_entry.predicate = *pred;

struct predicate has only 4 byte alignment, but the copy is using ld8.

[Bug ipa/64192] [5 Regression] r218369 causes some regressions with -m32.

[jamborm at gcc dot gnu.org]

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

--- Comment #4 from Martin Jambor <jamborm at gcc dot gnu.org> ---
I'm still in the process of testing the following fix.  I believe it will solve
all these problems.  Sorry for such a stupid mistake:

https://gcc.gnu.org/ml/gcc-patches/2014-12/msg00507.html

[Bug ipa/64192] [5 Regression] r218369 causes some regressions with -m32.

[hjl.tools at gmail dot com]

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

H.J. Lu <hjl.tools at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hjl.tools at gmail dot com

--- Comment #5 from H.J. Lu <hjl.tools at gmail dot com> ---
r218369 also miscompiled SPEC CPU 2000 on Linux/i686 with LTO:

https://gcc.gnu.org/ml/gcc-testresults/2014-12/msg00552.html

[Bug ipa/64192] [5 Regression] r218369 causes some regressions with -m32.

[jamborm at gcc dot gnu.org]

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

--- Comment #6 from Martin Jambor <jamborm at gcc dot gnu.org> ---
Author: jamborm
Date: Fri Dec  5 18:14:37 2014
New Revision: 218433

URL: https://gcc.gnu.org/viewcvs?rev=218433&root=gcc&view=rev
Log:
2014-12-05  Martin Jambor  <mjambor@suse.cz>

    PR ipa/64192
    * ipa-prop.c (ipa_compute_jump_functions_for_edge): Convert alignment
    from bits to bytes after checking they are byte-aligned.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/ipa-prop.c

[Bug ipa/64192] [5 Regression] r218369 causes some regressions with -m32.

[jamborm at gcc dot gnu.org]

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

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

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

--- Comment #7 from Martin Jambor <jamborm at gcc dot gnu.org> ---
OK, hopefully this is all fixed.  Please check and reopen if not.  Sorry for
the breakage.

[Bug ipa/64192] [5 Regression] r218369 causes some regressions with -m32.

[ro at gcc dot gnu.org]

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

Rainer Orth <ro at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ro at gcc dot gnu.org

--- Comment #8 from Rainer Orth <ro at gcc dot gnu.org> ---
*** Bug 64197 has been marked as a duplicate of this bug. ***