Re: [PATCH] Fix byte size confusion in bswap pass

2014-09-01 Thread Jakub Jelinek
On Fri, Aug 29, 2014 at 02:51:57PM +0800, Thomas Preud'homme wrote: > 2014-08-29 Thomas Preud'homme > > * tree-ssa-math-opts.c (struct symbolic_number): Clarify comment about > the size of byte markers. > (do_shift_rotate): Fix confusion between host, target and marker byte >

[PATCH] Fix byte size confusion in bswap pass

2014-08-28 Thread Thomas Preud'homme
[CCing you Jakub as you are the one who raised this issue to me] The bswap pass deals with 3 possibly different byte size: host, target and the size a byte marker in the symbolic_number structure [1]. However, right now the code mixes the three sizes. This works in practice as the pass is only e