Re: [PATCH] Remove bswap STRICT_ALING target limitation

2014-06-30 Thread Rainer Orth
Richard Biener richard.guent...@gmail.com writes:

 On June 27, 2014 2:50:33 PM CEST, Rainer Orth r...@cebitec.uni-bielefeld.de
 wrote:
Richard Biener rguent...@suse.de writes:

 The following patch enables bswap-from-load for STRICT_ALIGNMENT
 targets when the load is aligned instead of unconditionally
 (which probably was not intended).

 (note to self: we should fully transition to use
SLOW_UNALIGNED_ACCESS)

 Bootstrapped and tested on x86_64-unknown-linux-gnu.

 Richard.

 2014-06-27  Richard Biener  rguent...@suse.de

 * tree-ssa-math-opts.c (bswap_replace): Fix
 SLOW_UNALIGNED_ACCESS test to only apply to unaligned object.

This should fix PR bootstrap/61320.

 That would be odd.

You were right, of course: I should have read more closely.  Anyway, I'm
again back to bootstrap land until Thomas comes up with a real patch.

Rainer

-- 
-
Rainer Orth, Center for Biotechnology, Bielefeld University


Re: [PATCH] Remove bswap STRICT_ALING target limitation

2014-06-27 Thread Rainer Orth
Richard Biener rguent...@suse.de writes:

 The following patch enables bswap-from-load for STRICT_ALIGNMENT
 targets when the load is aligned instead of unconditionally
 (which probably was not intended).

 (note to self: we should fully transition to use SLOW_UNALIGNED_ACCESS)

 Bootstrapped and tested on x86_64-unknown-linux-gnu.

 Richard.

 2014-06-27  Richard Biener  rguent...@suse.de

   * tree-ssa-math-opts.c (bswap_replace): Fix
   SLOW_UNALIGNED_ACCESS test to only apply to unaligned object.

This should fix PR bootstrap/61320.

Rainer

-- 
-
Rainer Orth, Center for Biotechnology, Bielefeld University


Re: [PATCH] Remove bswap STRICT_ALING target limitation

2014-06-27 Thread Richard Biener
On June 27, 2014 2:50:33 PM CEST, Rainer Orth r...@cebitec.uni-bielefeld.de 
wrote:
Richard Biener rguent...@suse.de writes:

 The following patch enables bswap-from-load for STRICT_ALIGNMENT
 targets when the load is aligned instead of unconditionally
 (which probably was not intended).

 (note to self: we should fully transition to use
SLOW_UNALIGNED_ACCESS)

 Bootstrapped and tested on x86_64-unknown-linux-gnu.

 Richard.

 2014-06-27  Richard Biener  rguent...@suse.de

  * tree-ssa-math-opts.c (bswap_replace): Fix
  SLOW_UNALIGNED_ACCESS test to only apply to unaligned object.

This should fix PR bootstrap/61320.

That would be odd.

Richard.

   Rainer




Re: [PATCH] Remove bswap STRICT_ALING target limitation

2014-06-27 Thread Rainer Orth
Richard Biener richard.guent...@gmail.com writes:

 2014-06-27  Richard Biener  rguent...@suse.de

 * tree-ssa-math-opts.c (bswap_replace): Fix
 SLOW_UNALIGNED_ACCESS test to only apply to unaligned object.

This should fix PR bootstrap/61320.

 That would be odd.

It's almost exactly the workaround patch you've provided in that PR and
which I had to use ever since.  I just noticed because that didn't apply
anymore.  Currently running a bootstrap; we'll see how that goes.

Rainer

-- 
-
Rainer Orth, Center for Biotechnology, Bielefeld University