> On Apr 6, 2017, at 9:26 AM, Bill Schmidt <[email protected]> wrote:
>
>
>> On Apr 6, 2017, at 9:19 AM, Bill Schmidt <[email protected]> wrote:
>>
>>
>>> On Apr 6, 2017, at 9:04 AM, Richard Biener <[email protected]>
>>> wrote:
>>>
>>> On Thu, Apr 6, 2017 at 1:28 PM, Nathan Sidwell <[email protected]> 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
>>>
>>
>
