Richard Guenther <richard.guent...@gmail.com> writes:
> On Tue, Apr 12, 2011 at 3:28 PM, Richard Sandiford
> <richard.sandif...@linaro.org> wrote:
>> This patch just reindents part of vectorizable_load and vectorizable_store
>> so that the main diff is easier to read.  It also CSEs the element type,
>> which seemed better than breaking the long lines.
>>
>> I've included both the real diff and a -b version.
>>
>> Tested on x86_64-linux-gnu and arm-linux-gnueabi.  OK to install?
>
> CSEing element type is ok, but please don't install patches (separately)
> that introduce if (1)s.  I suppose this patch is to make followups smaller?

Yeah, patch 5 was pretty unreadable otherwise.

Richard

Reply via email to