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: Combiner fixes

From: Bernd Schmidt <bernds at codesourcery dot com>
To: "H.J. Lu" <hjl dot tools at gmail dot com>
Cc: Paolo Bonzini <bonzini at gnu dot org>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Earnshaw <rearnsha at arm dot com>
Date: Wed, 04 Aug 2010 18:30:30 +0200
Subject: Re: Combiner fixes
References: <4C572CA0.3040802@codesourcery.com> <4C57C416.70504@gnu.org> <4C582BD8.3080306@codesourcery.com> <AANLkTinkfnGVRHAQrEXK9gns6denDthS6zt38jn7f3w+@mail.gmail.com>

On 08/04/2010 05:39 PM, H.J. Lu wrote:
> This caused:
> 
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45182

This seems to be a latent bug.  We call make_compound_operation on

(plus:V2DI (ashift:V2DI (reg:V2DI 137)
        (const_int 2 [0x2]))
    (reg:V2DI 145))

and try to convert the shift into a multiplication, and
trunc_int_for_mode fails when asked to do something with V2DImode.

I think it's best not to do any of that for anything but plain integer
modes.  Ok if tests pass?

Bernd

Attachment: compoundop-fix.diff
Description: Text document

Follow-Ups:
- Re: Combiner fixes
  - From: H.J. Lu

References:
- Combiner fixes
  - From: Bernd Schmidt
- Re: Combiner fixes
  - From: Paolo Bonzini
- Re: Combiner fixes
  - From: Bernd Schmidt
- Re: Combiner fixes
  - From: H.J. Lu

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