[Bug c++/19144] New: SPARC backend bug

[Bug c++/19144] New: SPARC backend bug

[bugzilla-gcc at thewrittenword dot com]

We built qt-3.3.2 and kde-3.3.2 using gcc-3.4.3. We did the same on HP-UX 11.x,
Redhat Linux, and Redhat Enterprise Linux.
  http://lists.kde.org/?l=kde-devel&m=110384477320197&w=2
     
On Solaris/SPARC, we're seeing a problem. None of the other platforms have a
problem.

I was able to work around the bug in the above message with the diff below to
one of the KDE source files. Unfortunately, I cannot isolate this to a small
test case. I'm building on gax as follows:
  g++ -DHAVE_CONFIG_H -I. -I. -I../.. -I../../dcop -I../../kio/kssl
  -I../.. -I./.. -I./../kssl -I../kssl -I./../../interfaces -I../../dcop
  -I../../libltdl -I../../kdefx -I../../kdecore -I../../kdeui
  -I../../kio -I../../kio/kio -I../../kio/kfile -I../..
  -I/opt/TWWfsw/libqt33/include/gcc34 -I/opt/TWWfsw/kde33/include
  -I/opt/TWWfsw/bzip2/include -I/opt/TWWfsw/jpeg/include
  -I/opt/TWWfsw/libpcre44/include -I/opt/TWWfsw/libpng12/include
  -I/opt/TWWfsw/libxslt11/include -I/opt/TWWfsw/libxml26/include
  -I/opt/TWWfsw/libtiff35/include -I/opt/TWWfsw/zlib11/include
  -I/opt/TWWfsw/libart23/include -I/opt/TWWfsw/fcpackage22/include
  -I/opt/TWWfsw/libaudiofile02/include -I/opt/TWWfsw/cups1120r/include
  -I/opt/TWWfsw/libopenssl097/include -D_LARGEFILE64_SOURCE
  -DQT_THREAD_SUPPORT -D_REENTRANT -D_POSIX_PTHREAD_SEMANTICS
  -DUSE_SOLARIS -DSVR4 -DNDEBUG -DNO_DEBUG -isystem /usr/openwin/include
  -g -finline-functions -mcpu=v8 -fno-exceptions -fno-check-new
  -fno-common -DQT_CLEAN_NAMESPACE -DQT_NO_ASCII_CAST -DQT_NO_STL
  -DQT_NO_COMPAT -DQT_NO_TRANSLATION -c kurlcompletion.cpp

--- kdelibs-3.3.2/kio/kio/kurlcompletion.cpp.orig       Thu Dec 23 17:56:26 2004
+++ kdelibs-3.3.2/kio/kio/kurlcompletion.cpp    Thu Dec 23 19:11:59 2004
@@ -225,6 +225,7 @@

                // Loop through all directory entries

+int i;
                struct dirent dirPosition;
                struct dirent *dirEntry = 0;
                while ( !terminationRequested() &&

-- 
           Summary: SPARC backend bug
           Product: gcc
           Version: 3.4.3
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: bugzilla-gcc at thewrittenword dot com
                CC: gcc-bugs at gcc dot gnu dot org
 GCC build triplet: sparc-sun-solaris2.8
  GCC host triplet: sparc-sun-solaris2.8
GCC target triplet: sparc-sun-solaris2.8


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

[Bug c++/19144] SPARC backend bug

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 01:54 -------
The patch below, somewhat similar to the patch in the original message with the
workaround, does *not* work. The only difference between the two is where "int
i;" is placed.

--- kio/kio/kurlcompletion.cpp.orig     Thu Dec 23 17:56:26 2004
+++ kio/kio/kurlcompletion.cpp  Thu Dec 23 19:24:10 2004
@@ -226,6 +226,7 @@
                // Loop through all directory entries

                struct dirent dirPosition;
+int i;
                struct dirent *dirEntry = 0;
                while ( !terminationRequested() &&
                        ::readdir_r( dir, &dirPosition, &dirEntry ) == 0 &&
dirEntry )


-- 


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

[Bug c++/19144] SPARC backend bug

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 01:55 -------
Created an attachment (id=7815)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=7815&action=view)
.ii file using the original workaround patch


-- 


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

[Bug c++/19144] SPARC backend bug

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 01:56 -------
Created an attachment (id=7816)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=7816&action=view)
.s file using the original workaround patch


-- 


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

[Bug c++/19144] SPARC backend bug

[bugzilla-gcc at thewrittenword dot com]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #7815|text/plain                  |application/octet-stream
          mime type|                            |
Attachment #7815 is|1                           |0
              patch|                            |


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

[Bug target/19144] SPARC back-end bug

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-12-24 01:57 -------
This might not be a sparc back-end bug but a bug in the KDE, they might be overflowing the stack or 
something else.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|c++                         |target
           Keywords|                            |wrong-code
            Summary|SPARC backend bug           |SPARC back-end bug


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

[Bug target/19144] SPARC back-end bug

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 01:58 -------
Created an attachment (id=7817)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=7817&action=view)
.ii file using patch in comment #1


-- 


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

[Bug target/19144] SPARC back-end bug

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 01:59 -------
Created an attachment (id=7818)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=7818&action=view)
.s file using patch in comment #1


-- 


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

[Bug target/19144] SPARC back-end bug

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 02:03 -------
Ok, I ran diff on both .s files (id #7816 v id #7818) and found the following:
  @@ -199,8 +199,8 @@
          cmp     %g1, 0
          bne     .LL9
          nop
  -       add     %fp, -80, %g1
  -       add     %fp, -84, %o5
* +       add     %fp, -72, %g1
  +       add     %fp, -80, %o5
          ld      [%fp-20], %o0
          mov     %g1, %o1
          mov     %o5, %o2

If you review the diff of the .s files between #7816 and #7818, the
numeric differences are the same except for the line with the '*'
above. I modified the .s file and changed -72 to -76 (diff of four
from -80 on the next line) and it worked (patch below)!

--- id7816/kurlcompletion.s        Thu Dec 23 19:20:59 2004
+++ kurlcompletion.s    Thu Dec 23 19:40:52 2004
@@ -199,7 +199,7 @@
        cmp     %g1, 0
        bne     .LL9
        nop
-       add     %fp, -72, %g1
+       add     %fp, -76, %g1
        ld      [%fp-20], %o0
        mov     %g1, %o1


-- 


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

[Bug target/19144] SPARC back-end bug

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 02:04 -------
Comment #7 should confirm this is a GCC bug.

-- 


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

[Bug target/19144] SPARC back-end bug

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-12-24 02:08 -------
(In reply to comment #8)
> Comment #7 should confirm this is a GCC bug.
Not really, it just showed where the problem is.  Can you try the mainline as there has been a couple of 
sparc bugs which have been fixed there.

-- 


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

[Bug target/19144] SPARC back-end bug

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 02:59 -------
I looked at the .s files more closely and comment #7 might be off. Offsets seem
to differ by either 4 or 8 not only 8 like I originally thought. I generated the
.s files on HP-UX and it had the same 4 or 8 offset difference.

I'll test mainline.

-- 


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

[Bug target/19144] miscompilation KDE 3.3.2

[ebotcazou at gcc dot gnu dot org]

------- Additional Comments From ebotcazou at gcc dot gnu dot org  2004-12-24 06:36 -------
This PR is a bit confusing.  Please attach the original preprocessed file and
post the modification to the original *.s file that appears to fix the bug.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ebotcazou at gcc dot gnu dot
                   |                            |org
            Summary|SPARC back-end bug          |miscompilation KDE 3.3.2


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

[Bug target/19144] miscompilation KDE 3.3.2

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 07:00 -------
Created an attachment (id=7820)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=7820&action=view)
Original pre-processed source


-- 


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

[Bug target/19144] miscompilation KDE 3.3.2

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 07:00 -------
Created an attachment (id=7821)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=7821&action=view)
Original pre-processed .s file


-- 


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

[Bug target/19144] miscompilation KDE 3.3.2

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 07:06 -------
Created an attachment (id=7822)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=7822&action=view)
Pre-processed source with fix (add "int i;")

Note: If the 'int i;' is placed after the 'struct dirent dirPosition;', this
solution fails.

-- 


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

[Bug target/19144] miscompilation KDE 3.3.2

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 07:07 -------
Created an attachment (id=7823)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=7823&action=view)
Pre-processed .s of kurlcompletion.ii with fix ("int i;" added)


-- 


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

[Bug target/19144] miscompilation KDE 3.3.2

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 08:45 -------
gcc-3.3.2 exhibits the same problem. Maybe I'm wrong?

-- 


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

[Bug target/19144] miscompilation KDE 3.3.2

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 17:18 -------
Cannot test mailing because of an ICE while building qt-3.3.2 with -g.

-- 


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

[Bug target/19144] miscompilation KDE 3.3.2

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 17:34 -------
Oops, "mailing" in comment #17 should be "mainline".

-- 


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

[Bug target/19144] miscompilation KDE 3.3.2

[bugzilla-gcc at thewrittenword dot com]

------- Additional Comments From bugzilla-gcc at thewrittenword dot com  2004-12-24 18:50 -------
Ok, found the real problem. I was wrong. This is not a GCC bug. It is a KDE bug.
On Solaris and IRIX, struct dirent does not contain space for the name of the
file. Tru64 UNIX, HP-UX, and GNU/Linux do. So, KDE was simply overwriting memory
not belonging to it. Hence, SIGBUS.

Sorry!

-- 


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

[Bug target/19144] miscompilation KDE 3.3.2

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-12-24 18:52 -------
So closing as invalid. thanks for looking into the bug.

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


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