Richard Guenther <[email protected]> writes:
> On Tue, Apr 12, 2011 at 3:28 PM, Richard Sandiford
> <[email protected]> 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