[Bug tree-optimization/46077] New: [4.6 regression] ICE in tree vectorization when compiling towns_audio.cpp from scummvm

[Bug tree-optimization/46077] New: [4.6 regression] ICE in tree vectorization when compiling towns_audio.cpp from scummvm

[matt at use dot net]

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

           Summary: [4.6 regression] ICE in tree vectorization when
                    compiling towns_audio.cpp from scummvm
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: tree-optimization
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: matt@use.net


Created attachment 22085
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=22085
bzip2 compressed preprocessor output of the source file that reproduced the ICE

When compiling scummvm with Ubuntu 10.10's gcc-snapshot package version of g++
(Ubuntu 20101004-0ubuntu1), there is an ICE that occurs that does not happen
with g++-4.4 (Ubuntu/Linaro 4.4.4-14ubuntu5) or g++-4.5 (Ubuntu/Linaro
4.5.1-7ubuntu2):


matt@matt-desktop:~/src/scummvm$ g++ -x c++ -O3 /tmp/towns_audio.cpp.pre 
sound/softsynth/fmtowns_pc98/towns_audio.cpp: In member function 'int
TownsAudioInterface::intf_pcmPlayEffect(__va_list_tag (&)[1])':
sound/softsynth/fmtowns_pc98/towns_audio.cpp:604:5: internal compiler error: in
vect_update_ivs_after_vectorizer, at tree-vect-loop-manip.c:1779

The pre-processed source file that reproduced the issue is attached.

[Bug tree-optimization/46077] [4.6 regression] ICE in tree vectorization when compiling towns_audio.cpp from scummvm

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.6.0

[Bug tree-optimization/46077] [4.6 regression] ICE in tree vectorization when compiling towns_audio.cpp from scummvm

[irar at il dot ibm.com]

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

Ira Rosen <irar at il dot ibm.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |irar at il dot ibm.com

--- Comment #1 from Ira Rosen <irar at il dot ibm.com> 2010-10-19 11:54:34 UTC ---
Maybe related to PR 45971?

[Bug tree-optimization/46077] [4.6 regression] ICE in tree vectorization when compiling towns_audio.cpp from scummvm

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2010.11.04 13:17:47
         Depends on|                            |45971
     Ever Confirmed|0                           |1

--- Comment #2 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-11-04 13:17:47 UTC ---
Reducing.

[Bug tree-optimization/46077] [4.6 regression] ICE in tree vectorization when compiling towns_audio.cpp from scummvm

[rguenth at gcc dot gnu.org]

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

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

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

--- Comment #3 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-11-04 13:24:38 UTC ---
Reduced testcase:

extern "C" {
    typedef __builtin_va_list __gnuc_va_list;
    typedef __gnuc_va_list va_list;
}
typedef unsigned char uint8;
typedef unsigned int uint32;
typedef signed char int8;
namespace Audio {
    class AudioStream {
    };
}
class TownsPC98_FmSynth : public Audio::AudioStream {
};
class TownsAudio_PcmChannel;
class TownsAudioInterface : public TownsPC98_FmSynth {
    int intf_pcmPlayEffect(va_list &args);
    TownsAudio_PcmChannel *_pcmChan;
};
class TownsAudio_PcmChannel {
    friend class TownsAudioInterface;
    void loadExtData(uint8 *buffer, uint32 size);
    int8 *extData;
};
class TownsAudio_WaveTable {
    friend class TownsAudioInterface;
    uint32 size;
};
int TownsAudioInterface::intf_pcmPlayEffect(va_list &args) {
    int chan = __builtin_va_arg(args,int);
    uint8 *data = __builtin_va_arg(args,uint8 *);
    TownsAudio_WaveTable w;
    TownsAudio_PcmChannel *p = &_pcmChan[chan];
    p->loadExtData(data + 32, w.size);
}
void TownsAudio_PcmChannel::loadExtData(uint8 *buffer, uint32 size) {
    int8 *src = (int8 *)buffer;
    int8 *dst = extData;
    for (uint32 i = 0; i < size; i++)
      *dst++ = *src & 0x80 ? (*src++ & 0x7f) : -*src++;
}

Indeed makes it look like a dup.

*** This bug has been marked as a duplicate of bug 45971 ***

[Bug tree-optimization/46077] [4.6 regression] ICE in tree vectorization when compiling towns_audio.cpp from scummvm

[matt at use dot net]

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

Matt Hargett <matt at use dot net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|DUPLICATE                   |

--- Comment #4 from Matt Hargett <matt at use dot net> 2010-11-17 22:10:58 UTC ---
The fix for the bug this was marked as a duplicate of does not fix the bug's
problem. Tested with the 4.6.0.20101116 build.

[Bug tree-optimization/46077] [4.6 regression] ICE in tree vectorization when compiling towns_audio.cpp from scummvm

[matt at use dot net]

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

Matt Hargett <matt at use dot net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |critical

[Bug tree-optimization/46077] [4.6 regression] ICE in tree vectorization when compiling towns_audio.cpp from scummvm

[pinskia at gcc dot gnu.org]

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

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |ice-on-valid-code
           Severity|critical                    |normal

[Bug tree-optimization/46077] [4.6 regression] ICE in tree vectorization when compiling towns_audio.cpp from scummvm

[hjl.tools at gmail dot com]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |NEW
                 CC|                            |matz at suse dot de

--- Comment #5 from H.J. Lu <hjl.tools at gmail dot com> 2010-11-19 05:23:06 UTC ---
It is caused by revision 163998:

http://gcc.gnu.org/ml/gcc-cvs/2010-09/msg00290.html

[Bug tree-optimization/46077] [4.6 regression] ICE in tree vectorization when compiling towns_audio.cpp from scummvm

[matz at gcc dot gnu.org]

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

--- Comment #6 from Michael Matz <matz at gcc dot gnu.org> 2010-11-19 20:56:33 UTC ---
Author: matz
Date: Fri Nov 19 20:56:27 2010
New Revision: 166958

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=166958
Log:
    PR tree-optimization/46077
    * tree-chrec.c (eq_evolutions_p): Accept some expressions.

testsuite/
    PR tree-optimization/46077
    * gcc.dg/vect/O3-pr46077.c: New testcase.

Added:
    trunk/gcc/testsuite/gcc.dg/vect/O3-pr46077.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree-chrec.c

[Bug tree-optimization/46077] [4.6 regression] ICE in tree vectorization when compiling towns_audio.cpp from scummvm

[matz at gcc dot gnu.org]

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

Michael Matz <matz at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |matz at gcc dot gnu.org
         Resolution|                            |FIXED

--- Comment #7 from Michael Matz <matz at gcc dot gnu.org> 2010-11-19 21:03:34 UTC ---
Fixed.