On Fri, Oct 3, 2014 at 6:46 AM, Ilya Tocar <tocarip.in...@gmail.com> wrote:
> On 02 Oct 07:41, H.J. Lu wrote:
>> On Thu, Oct 2, 2014 at 7:29 AM, Ilya Tocar <tocarip.in...@gmail.com> wrote:
>> > Hi,
>> >
>> > sizeof (long) == 4 on windows, so we should use long long as param type.
>> > Patch below does it.
>>
>> The same is true for x32. Can you add a testcase to show it
>> fails on x32 without the fix?
>>
>
> This could only be done with runtime test.
> I've had troubles running sde (emulator) on x32 enabled system,
> but replacing long long with int in intrinsic signature will cause
> adx-addcarryx64-2.c to fail under sde on 64 bits. I believe it will
> also fail on sde+{win,x32} or real hardware, when it's available.
>
Can we scan the assembly output for the wrong instruction?
--
H.J.
