Jakub Jelinek <ja...@redhat.com> writes:
> On Tue, Mar 05, 2019 at 09:26:19AM +0000, Richard Sandiford wrote:
>> LGTM, thanks.  Given the discussion, I think it would also be worth having
>> a comment explaining why we're doing this, something like:
>> 
>>   /* Avoid speculatively generating a stand-alone vector comparison
>>      on targets that might not support them.  Any target implementing
>>      conditional internal functions must support the same comparisons
>>      inside and outside a VEC_COND_EXPR.  */
>
> Ok, here is the patch updated with your comment.
>
> Can you please test it on aarch64 SVE?  I'll test it on x86_64/i686 later
> today (maybe powerpc64{,le} too).

OK, testing passed on SVE.

Thanks,
Richard

Reply via email to