[Bug c++/51381] New: Internal compiler error for arm target

[Bug c++/51381] New: Internal compiler error for arm target

[eric.valette at free dot fr]

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

             Bug #: 51381
           Summary: Internal compiler error for arm target
    Classification: Unclassified
           Product: gcc
           Version: 4.6.2
            Status: UNCONFIRMED
          Severity: major
          Priority: P3
         Component: c++
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: eric.valette@free.fr
              Host: Linux trimslice armv7l GNU/Linux arm-linux-gnueabihf
            Target: arm-linux-gnueabihf


Internal compiler error 

GUIInfoManager.cpp: In member function ‘CStdString
CGUIInfoManager::GetMusicLabel(int)’:
GUIInfoManager.cpp:3349:1: internal compiler error: in fp_immediate_constant,
at config/arm/arm.c:12546

[Bug c++/51381] Internal compiler error for arm target

[mikpe at it dot uu.se]

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

--- Comment #1 from Mikael Pettersson <mikpe at it dot uu.se> 2011-12-01 15:57:08 UTC ---
Please provide preprocessed source for the test case, the options used to
compile it, and how the compiler was configured.

[Bug c++/51381] Internal compiler error for arm target

[eric.valette at free dot fr]

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

--- Comment #2 from eric.valette at free dot fr 2011-12-01 17:08:49 UTC ---
I cannot attach the preprocessed file directly because its too biig. Will try
to zip it. I already attached it but get no error first time.

[Bug c++/51381] Internal compiler error for arm target

[eric.valette at free dot fr]

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

--- Comment #3 from eric.valette at free dot fr 2011-12-01 17:10:15 UTC ---
Created attachment 25968
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25968
bziped preprocessed file

[Bug c++/51381] Internal compiler error for arm target

[eric.valette at free dot fr]

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

--- Comment #4 from eric.valette at free dot fr 2011-12-01 17:20:38 UTC ---
Compilation option for the file

-g -O2 -Wall -Woverloaded-virtual -Wno-parentheses -fPIC -DPIC -D_REENTRANT
-D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -g -D_DEBUG -Wall -O2
-mfloat-abi=hard -mno-apcs-stack-check -Wno-psabi -Wa,-march=armv7a
-mtune=cortex-a9 -mfpu=vfpv3-d16 -mthumb-interwork

[Bug c++/51381] Internal compiler error for arm target

[eric.valette at free dot fr]

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

--- Comment #5 from eric.valette at free dot fr 2011-12-01 17:25:15 UTC ---
g++ -v
Using built-in specs.
COLLECT_GCC=g++
COLLECT_LTO_WRAPPER=/usr/lib/gcc/arm-linux-gnueabihf/4.6/lto-wrapper
Target: arm-linux-gnueabihf
Configured with: ../src/configure -v --with-pkgversion='Debian 4.6.2-5'
--with-bugurl=file:///usr/share/doc/gcc-4.6/README.Bugs
--enable-languages=c,c++,fortran,objc,obj-c++ --prefix=/usr
--program-suffix=-4.6 --enable-shared --enable-linker-build-id
--with-system-zlib --libexecdir=/usr/lib --without-included-gettext
--enable-threads=posix --with-gxx-include-dir=/usr/include/c++/4.6
--libdir=/usr/lib --enable-nls --enable-clocale=gnu --enable-libstdcxx-debug
--enable-libstdcxx-time=yes --enable-plugin --enable-objc-gc
--disable-sjlj-exceptions --with-arch=armv7-a --with-float=hard
--with-fpu=vfpv3-d16 --with-mode=thumb --enable-checking=release
--build=arm-linux-gnueabihf --host=arm-linux-gnueabihf
--target=arm-linux-gnueabihf
Thread model: posix
gcc version 4.6.2 (Debian 4.6.2-5)

[Bug target/51381] Internal compiler error for arm target

[pinskia at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|c++                         |target
           Severity|major                       |normal

[Bug c++/51381] Internal compiler error for arm target

[eric.valette at free dot fr]

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

eric.valette at free dot fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|target                      |c++
           Severity|normal                      |major

--- Comment #6 from eric.valette at free dot fr 2011-12-01 18:39:56 UTC ---
I guess I've been hit by the Emit-conditions-in-push_multi problem. I applied
the patch below after verifying ist upstream but have not yet rebuild gcc

http://old.nabble.com/ARM%3A-Emit-conditions-in-push_multi-td32379089.html#a32386142

[Bug target/51381] Internal compiler error for arm target

[pinskia at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|c++                         |target
           Severity|major                       |normal

[Bug target/51381] Internal compiler error for arm target

[eric.valette at free dot fr]

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

--- Comment #7 from eric.valette at free dot fr 2011-12-02 14:58:49 UTC ---
The line given in the error correspond to a gcc_unreachable call in the
fp_immediate_constant (rtx x) function???

[Bug target/51381] Internal compiler error for arm target

[eric.valette at free dot fr]

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

--- Comment #8 from eric.valette at free dot fr 2011-12-03 10:32:39 UTC ---
(In reply to comment #6)
> I guess I've been hit by the Emit-conditions-in-push_multi problem. I applied
> the patch below after verifying ist upstream but have not yet rebuild gcc
> 
> http://old.nabble.com/ARM%3A-Emit-conditions-in-push_multi-td32379089.html#a32386142

Does not solve the problem.

[Bug target/51381] Internal compiler error for arm target

[mikpe at it dot uu.se]

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

--- Comment #9 from Mikael Pettersson <mikpe at it dot uu.se> 2011-12-03 20:45:02 UTC ---
Created attachment 25983
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25983
reduced test case in C

The big test case easily reduced to the following snippet:

void round_int(void)
{
    __asm__ __volatile__("vmov.F64 d1,%0" : : "Dv"(0.5f) : "d1");
}

Compiled with -march=armv7-a -mtune=cortex-a9 -mfloat-abi=hard -mfpu=vfpv3-d16
it ICEs trunk @ r181973 as follows:

pr51381.c: In function 'round_int':
pr51381.c:6:1: internal compiler error: in fp_immediate_constant, at
config/arm/arm.c:13418

Current gcc 4.6 and 4.5 also ICE in fp_immediate_constant().  gcc-4.4 rejects
the command line options (sorry, unimplemented: -mfloat-abi=hard and VFP).

[Bug target/51381] Internal compiler error for arm target

[eric.valette at free dot fr]

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

--- Comment #10 from eric.valette at free dot fr 2011-12-03 23:49:42 UTC ---
Then at least I know how to fix the bug. Thanks a lot for your help. I will use
the C definition instead of inline asm. BTW I have no clue on arm assembler. Is
the asm code correct?

[Bug target/51381] Internal compiler error for arm target

[ramana at gcc dot gnu.org]

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

Ramana Radhakrishnan <ramana at gcc dot gnu.org> changed:

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

--- Comment #11 from Ramana Radhakrishnan <ramana at gcc dot gnu.org> 2011-12-05 17:23:36 UTC ---
I don't think this is ICE on valid code. The compiler at the minute doesn't
generate unified syntax for VFP instructions - therefore expecting the
instructions of that form to work correctly is wrong. 

I think the correct way of writing this is 

void round_int(void)
{
    __asm__ __volatile__("fconstd d1,%G0" : : "Dv"(0.5f) : "d1");
}

And then it would work. 

We really do need to transition to unified syntax for VFP instructions. 

cheers
Ramana

[Bug target/51381] Internal compiler error for arm target

[ramana at gcc dot gnu.org]

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

Ramana Radhakrishnan <ramana at gcc dot gnu.org> changed:

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

--- Comment #12 from Ramana Radhakrishnan <ramana at gcc dot gnu.org> 2012-01-11 12:13:50 UTC ---
As per comment #11

[Bug target/51381] Internal compiler error for arm target

[eric.valette at free dot fr]

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

--- Comment #13 from eric.valette at free dot fr 2012-01-11 12:22:57 UTC ---
Even if the code is invalid, gcc should not crash with a ICE but notify that
the asm is invalid.