On Thu, Jan 15, 2015 at 11:06:38AM -0500, David Edelsohn wrote:
> On Wed, Jan 14, 2015 at 8:14 PM, Segher Boessenkool
> <seg...@kernel.crashing.org> wrote:
> > This fixes 141 FAILs.
> >
> > -mpowerpc64 does not change the ABI, but default_scalar_mode_supported_p
> > does not know that, and allows TImode for -m32 -mpowerpc64.

> Would you please add a comment that efficient TImode arithmetic
> requires carry to explain the TARGET_32BIT relationship?

I expanded the comment to:

  /* -m32 does not support TImode.  This is the default, from
     default_scalar_mode_supported_p.  For -m32 -mpowerpc64 we want the
     same ABI as for -m32.  But default_scalar_mode_supported_p allows
     integer modes of precision 2 * BITS_PER_WORD, which matches TImode
     for -mpowerpc64.  */

because the reasons why we do or do not suppoort TImode with -m32 are
pretty much irrelevant here.  Hope that's okay.

> We can think about providing TImode logical operations in 32 bit, in
> the long run.

Generic code would handle that just fine already, AFAICS.  We just do
not allow TImode at all (like most (all?) other 32-bit targets).


Segher

Reply via email to