From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-return-103157-listarch-gcc=gcc.gnu.org@gcc.gnu.org>
Received: (qmail 25507 invoked by alias); 23 Sep 2004 15:11:08 -0000
Mailing-List: contact gcc-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Archive: <http://gcc.gnu.org/ml/gcc/>
List-Post: <mailto:gcc@gcc.gnu.org>
List-Help: <http://gcc.gnu.org/ml/>
Sender: gcc-owner@gcc.gnu.org
Received: (qmail 25492 invoked from network); 23 Sep 2004 15:11:05 -0000
Received: from unknown (HELO geburtsjahr.neunzehnhunderteinundachtzig.de) (217.160.92.50)
  by sourceware.org with SMTP; 23 Sep 2004 15:11:05 -0000
Received: from bauernhof (pD9EBE115.dip.t-dialin.net [217.235.225.21])
	by geburtsjahr.neunzehnhunderteinundachtzig.de (Postfix) with ESMTP
	id 7982C8FC82; Thu, 23 Sep 2004 17:12:25 +0200 (CEST)
Received: from [192.168.1.6] (unknown [192.168.1.6])
	by bauernhof (Postfix) with ESMTP
	id 96A9B63249; Thu, 23 Sep 2004 17:10:35 +0200 (CEST)
Message-ID: <4152E765.1060702@physik.uni-muenchen.de>
Date: Thu, 23 Sep 2004 16:31:00 -0000
From: =?ISO-8859-1?Q?Tobias_Schl=FCter?= <tobias.schlueter@physik.uni-muenchen.de>
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.2) Gecko/20040803
MIME-Version: 1.0
To: =?ISO-8859-1?Q?Tobias_Schl=FCter?= <tobias.schlueter@physik.uni-muenchen.de>
Cc: Richard Henderson <rth@redhat.com>, gcc@gcc.gnu.org,
	GCC Fortran mailing list <fortran@gcc.gnu.org>
Subject: Re: Your patch causes regression in the Fortran frontend
References: <41505234.2070602@physik.uni-muenchen.de>
In-Reply-To: <41505234.2070602@physik.uni-muenchen.de>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit
X-SW-Source: 2004-09/txt/msg01344.txt.bz2

Tobias Schlüter wrote:
> The patch you submitted on Sept 13th,
> <http://gcc.gnu.org/ml/gcc-cvs/2004-09/msg00581.html>, with ChangeLog entry
> 
> 	PR tree-opt/10528
> 	* tree-inline.c (copy_body_r): Recompute bits for ADDR_EXPR,
> 	after copying its argument.
> 
> causes the following regressions in the Fortran frontend on i686-pc-linux:
...
> I.e. all of these disappear if the above change is backed out.
> 

Minimal testcase which breaks with your patch, but works if your patch is removed:
-------------------
program main
call f
contains
  subroutine f
    external g
    call g(5) ! changing 5 to i fixes ICE
  end subroutine f
end program main
-----------------

The problem seems to be the call to an external function from a contained
function which is being inlined.

Error message (this is from yesterday's CVS, but I didn't see any related
changes on gcc-cvs):
[tobi@marktplatz tests]$ gfortran cont.f90 -O2
unhandled expression in get_expr_operands():
 <const_decl 0xf6db9244
    type <integer_type 0xf6e23488 int4 public SI
        size <integer_cst 0xf6e20408 constant invariant 32>
        unit size <integer_cst 0xf6e20138 constant invariant 4>
        align 32 symtab 0 alias set -1 precision 32 min <integer_cst
0xf6e203d8 -2147483648> max <integer_cst 0xf6e203f0 2147483647>
        pointer_to_this <pointer_type 0xf6e0f000>>
    addressable VOID file cont.f90 line 6
    align 1 context <function_decl 0xf6db90e8 f> initial <integer_cst
0xf6db8108 5>>

cont.f90: In function 'MAIN__':
cont.f90:2: internal compiler error: internal error
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.

Hope this helps,
- Tobi