This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: A few simple DImode improvements


On 06/29/2010 01:34 AM, Bernd Schmidt wrote:
> On 06/29/2010 01:03 AM, Ian Lance Taylor wrote:
>> I see what you're getting at.  It's unlikely, but it does happen in code
>> which examines the bits of a floating point number via a union or using
>> -fno-strict-aliasing and a type cast.  So if that code works correctly
>> if perhaps less efficiently, then I agree that checking that the mode
>> class is MODE_INT should work OK here.
> 
> Hmm.  Looking at this again now, aren't we already doing that?
> (resolve_reg_p checks whether DEST is a CONCAT)
> 
>   if (!can_decompose_p (dest)
>       || (side_effects_p (dest) && !pushing)
>       || (!SCALAR_INT_MODE_P (dest_mode)
> 	  && !resolve_reg_p (dest)
> 	  && !resolve_subreg_p (dest)))

In other words, how about the following patch?  Bootstrapped and
regression tested on i686-linux.


Bernd

Attachment: ls2.diff
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]