[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gcc-fortran
Subject:    Re: [fortran patch] Don't use TREE_LISTs for storing arguments to intrinsic functions
From:       FX Coudert <fxcoudert () gmail ! com>
Date:       2007-05-29 5:59:50
Message-ID: 390DE704-173F-4F66-87A3-73A9E46DD40A () gmail ! com
[Download RAW message or body]

> Do you happen to have a testcase that highlighted this problem?

Not that I know of. I think it was generating code that is valid, but  
inefficient. Instead of translating MAX(A,B) into
     if (A > B)
       MAX = A;
     else
       MAX = B;
your patch was generating
     if (A > A)
       MAX = A;
     else
       MAX = A;
     if (B > MAX)
       MAX = B;

This yields the right result, but is inefficient. Thus, I don't have  
a testcase.

FX

[prev in list] [next in list] [prev in thread] [next in thread]