[Bug c++/38253] New: g++.dg/ipa/iinline-1.C scan-ipa-dump inline fails on powerpc-apple-darwin9

[Bug c++/38253] New: g++.dg/ipa/iinline-1.C scan-ipa-dump inline fails on powerpc-apple-darwin9

[howarth at nitro dot med dot uc dot edu]

The test case for g++.dg/ipa/iinline-1.C scan-ipa-dump inline fails on
powerpc-apple-darwin9 as follows in current gcc trunk...

Executing on host:
/sw/src/fink.build/gcc44-4.3.999-20081120/darwin_objdir/gcc/testsuite/g++/../../g++
-B/sw/src/fink.build/gcc44-4.3.999-20081120/darwin_objdir/gcc/testsuite/g++/../../
/sw/src/fink.build/gcc44-4.3.999-20081120/gcc-4.4-20081120/gcc/testsuite/g++.dg/ipa/iinline-1.C
 -nostdinc++
-I/sw/src/fink.build/gcc44-4.3.999-20081120/darwin_objdir/powerpc-apple-darwin9/libstdc++-v3/include/powerpc-apple-darwin9
-I/sw/src/fink.build/gcc44-4.3.999-20081120/darwin_objdir/powerpc-apple-darwin9/libstdc++-v3/include
-I/sw/src/fink.build/gcc44-4.3.999-20081120/gcc-4.4-20081120/libstdc++-v3/libsupc++
-I/sw/src/fink.build/gcc44-4.3.999-20081120/gcc-4.4-20081120/libstdc++-v3/include/backward
-I/sw/src/fink.build/gcc44-4.3.999-20081120/gcc-4.4-20081120/libstdc++-v3/testsuite/util
-fmessage-length=0  -O3 -fdump-ipa-inline -fno-early-inlining -fpie  -S  -o
iinline-1.s    (timeout = 300)
PASS: g++.dg/ipa/iinline-1.C (test for excess errors)
FAIL: g++.dg/ipa/iinline-1.C scan-ipa-dump inline "String::funcOne[^\n]*inline
copy in int main"


-- 
           Summary: g++.dg/ipa/iinline-1.C scan-ipa-dump inline fails on
                    powerpc-apple-darwin9
           Product: gcc
           Version: 4.4.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: howarth at nitro dot med dot uc dot edu
 GCC build triplet: powerpc-apple-darwin9
  GCC host triplet: powerpc-apple-darwin9
GCC target triplet: powerpc-apple-darwin9


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38253

[Bug c++/38253] g++.dg/ipa/iinline-1.C scan-ipa-dump inline fails on powerpc-apple-darwin9

[howarth at nitro dot med dot uc dot edu]

------- Comment #1 from howarth at nitro dot med dot uc dot edu  2008-11-24 18:52 -------
Created an attachment (id=16763)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16763&action=view)
assembly file for  g++.dg/ipa/iinline-1.C on powerpc-apple-darwin9


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38253

[Bug c++/38253] g++.dg/ipa/iinline-1.C scan-ipa-dump inline fails on powerpc-apple-darwin9

[ubizjak at gmail dot com]

------- Comment #2 from ubizjak at gmail dot com  2008-11-24 23:07 -------
(In reply to comment #1)
> Created an attachment (id=16763)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16763&action=view) [edit]
> assembly file for  g++.dg/ipa/iinline-1.C on powerpc-apple-darwin9

Please attach an _.inline dump (produced with -fdump-ipa-inline).


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38253

[Bug c++/38253] g++.dg/ipa/iinline-1.C scan-ipa-dump inline fails on powerpc-apple-darwin9

[howarth at nitro dot med dot uc dot edu]

------- Comment #3 from howarth at nitro dot med dot uc dot edu  2008-11-25 01:08 -------
Created an attachment (id=16765)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16765&action=view)
inline file for iinline-1.C on powerpc-apple-darwin9


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38253

[Bug c++/38253] g++.dg/ipa/iinline-1.C scan-ipa-dump inline fails on powerpc-apple-darwin9

[janis at gcc dot gnu dot org]

------- Comment #4 from janis at gcc dot gnu dot org  2008-12-02 00:00 -------
This test also fails on powerpc64-unknown-linux-gnu for both -m32 and -m64.  It
passed on 2008-08-24 (r138089) and failed on 2008-08-25 (r138121).  Let me know
if you'd like me to run a regression hunt and/or provide dump files.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38253

[Bug c++/38253] [4.4 regression] g++.dg/ipa/iinline-1.C scan-ipa-dump inline fails on powerpc

[janis at gcc dot gnu dot org]

-- 

janis at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
  GCC build triplet|powerpc-apple-darwin9       |
   GCC host triplet|powerpc-apple-darwin9       |
 GCC target triplet|powerpc-apple-darwin9       |powerpc*-*-*
   Last reconfirmed|0000-00-00 00:00:00         |2008-12-09 23:08:28
               date|                            |
            Summary|g++.dg/ipa/iinline-1.C scan-|[4.4 regression]
                   |ipa-dump inline fails on    |g++.dg/ipa/iinline-1.C scan-
                   |powerpc-apple-darwin9       |ipa-dump inline fails on
                   |                            |powerpc
   Target Milestone|---                         |4.4.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38253

[Bug c++/38253] [4.4 regression] g++.dg/ipa/iinline-1.C scan-ipa-dump inline fails on powerpc

[jakub at gcc dot gnu dot org]

------- Comment #5 from jakub at gcc dot gnu dot org  2008-12-10 13:13 -------
I guess the primary difference between e.g. x86_64 or i386 and powerpc is
during gimplification, the pointer-to-member initialization is done as:
    D.1804.__pfn = funcOne;
    D.1804.__delta = 0;
    i.1 = docalling (D.1804);
on i386/x86_64, but as:
  static struct 
{
  String:: * __pfn;
  long int __delta;
} C.1 = {.__pfn=funcOne, .__delta=0};
...
    D.1807 = C.1;
    i.2 = docalling (D.1807);

and later on esra even makes it harder to find out what the function is.

This is gimplify_init_constructor, where:
if (size > 0 && !can_move_by_pieces (size, align))
(for -m64 with size 16, align 64, for -m32 with size 8, align 32)
on powerpc is true and so a temporary is created, on i386/x86_64
can_move_by_pieces returns 1 because MOVE_RATIO is larger and so it is
initialized piece-wise.

In this case (2 integral field ctor, one entry is 0, one is an address) I doubt
it makes sense on any arch to do the temporary (if that would be true, then
just the constant should be forced into memory by mov[sd]i expander).
can_move_by_pieces doesn't make much sense either, we are deciding whether to
initialize piece-wise or not, for piece-wise initialization we want to see if
storing the constants into memory isn't too expensive (so more like
can_store_by_pieces with appropriate callback).


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38253

[Bug c++/38253] [4.4 regression] g++.dg/ipa/iinline-1.C scan-ipa-dump inline fails on powerpc

[jakub at gcc dot gnu dot org]

------- Comment #6 from jakub at gcc dot gnu dot org  2008-12-10 13:16 -------
BTW, the testcase has been added in r138092, by passed in r138089 you mean
just that you didn't see FAIL (as the test didn't exist)?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38253

[Bug c++/38253] [4.4 regression] g++.dg/ipa/iinline-1.C scan-ipa-dump inline fails on powerpc

[jakub at gcc dot gnu dot org]

------- Comment #7 from jakub at gcc dot gnu dot org  2008-12-11 09:16 -------
Subject: Bug 38253

Author: jakub
Date: Thu Dec 11 09:15:17 2008
New Revision: 142674

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=142674
Log:
        PR c++/38253
        * gimplify.c (gimplify_init_constructor): Don't force constructor
        into memory if there is just one nonzero element.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/gimplify.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38253

[Bug c++/38253] [4.4 regression] g++.dg/ipa/iinline-1.C scan-ipa-dump inline fails on powerpc

[jakub at gcc dot gnu dot org]

------- Comment #8 from jakub at gcc dot gnu dot org  2008-12-11 09:17 -------
Fixed.


-- 

jakub at gcc dot gnu dot org changed:

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


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38253