> On Apr 6, 2017, at 9:26 AM, Bill Schmidt <wschm...@linux.vnet.ibm.com> wrote:
>
>
>> On Apr 6, 2017, at 9:19 AM, Bill Schmidt <wschm...@linux.vnet.ibm.com> wrote:
>>
>>
>>> On Apr 6, 2017, at 9:04 AM, Richard Biener <richard.guent...@gmail.com>
>>> wrote:
>>>
>>> On Thu, Apr 6, 2017 at 1:28 PM, Nathan Sidwell <nat...@acm.org> wrote:
>>>> Let's try this one then.
>>
>> Nathan's patch regstraps cleanly. I'll try Richard's variant (dropping the
>> if test below) now.
As expected, this version passes regstrap as well.
Bill
>
> FYI, the test case should also include:
>
> // { dg-require-effective-target powerpc_altivec_ok }
>
> to avoid problems on targets without AltiVec support.
>
> Bill
>>
>> Bill
>>>
>>> I'd call this
>>>
>>> + if (result == TYPE_CANONICAL (result))
>>> + /* Copy so we don't give the canonical type a name. */
>>> + result = build_variant_type_copy (result);
>>>
>>> premature optimization -- I wonder if anything breaks if you always copy?
>>> (that is, I expect result is always the canonical type here?)
>>>
>>> Richard.
>>>
>>>> nathan
>>>>
>>>> --
>>>> Nathan Sidwell
>>>
>>
>
