search

Terraspace / UASM

UASM - Macro Assembler
http://www.terraspace.co.uk/uasm.html
Other
220 stars 49 forks source link

Minor AVX512 issue #32

Closed gwoltman closed 8 years ago

gwoltman commented 8 years ago

vmovdqa zmm1, ZMMWORD PTR abc

does not assemble, but

vmovdqa zmm1, YMMWORD PTR abc

does

john-terraspace commented 8 years ago

Thanks for spotting these issues, we’ll investigate and sort them out!

From: gwoltman [mailto:notifications@github.com] Sent: 18 September 2016 04:04 AM To: Terraspace/HJWasm HJWasm@noreply.github.com Subject: [Terraspace/HJWasm] Minor AVX512 issue (#32)

vmovdqa zmm1, ZMMWORD PTR abc

does not assemble, but

vmovdqa zmm1, YMMWORD PTR abc

does

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Terraspace/HJWasm/issues/32 , or mute the thread https://github.com/notifications/unsubscribe-auth/AQGQVC4Pld4WB5oBol4AUjpHGvkflc7uks5qrKoYgaJpZM4J_yFi .

john-terraspace commented 8 years ago

Can you send me the data declaration for abc ?

I’ve tried a few different combinations of this and it appears to work for me.

From: gwoltman [mailto:notifications@github.com] Sent: 18 September 2016 04:04 AM To: Terraspace/HJWasm HJWasm@noreply.github.com Subject: [Terraspace/HJWasm] Minor AVX512 issue (#32)

vmovdqa zmm1, ZMMWORD PTR abc

does not assemble, but

vmovdqa zmm1, YMMWORD PTR abc

does

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Terraspace/HJWasm/issues/32 , or mute the thread https://github.com/notifications/unsubscribe-auth/AQGQVC4Pld4WB5oBol4AUjpHGvkflc7uks5qrKoYgaJpZM4J_yFi .

gwoltman commented 8 years ago

Hi,

Thanks for looking into these issues.

"abc" was really ZMM_BITS30 declared as follows:

ZMM_BITS30 DD 3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0

Best regards, George Woltman

On Sun, Sep 18, 2016 at 5:59 AM, John Hankinson notifications@github.com wrote:

Can you send me the data declaration for abc ?

I’ve tried a few different combinations of this and it appears to work for me.

From: gwoltman [mailto:notifications@github.com] Sent: 18 September 2016 04:04 AM To: Terraspace/HJWasm HJWasm@noreply.github.com Subject: [Terraspace/HJWasm] Minor AVX512 issue (#32)

vmovdqa zmm1, ZMMWORD PTR abc

does not assemble, but

vmovdqa zmm1, YMMWORD PTR abc

does

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ Terraspace/HJWasm/issues/32 , or mute the thread https://github.com/ notifications/unsubscribe-auth/AQGQVC4Pld4WB5oBol4AUjpHGvkflc 7uks5qrKoYgaJpZM4J_yFi .

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Terraspace/HJWasm/issues/32#issuecomment-247838591, or mute the thread https://github.com/notifications/unsubscribe-auth/AVPFMwU7daXWavrNY-b5J6Fl6oEp3jOVks5qrQuEgaJpZM4J_yFi .

john-terraspace commented 8 years ago

Hi,

We have implemented some changes for this which will be ready either later today or tomorrow.

There are some notes however:

Technically, there is no instruction vmovdqa zmm(n), …

Rather only vmovdqa32 or vmovdqa64.

As such we’ve added automatic conversion of the instruction, so vmovdqa zmm(n),MEM with automatically convert to vmovdqa32/64.

The ensure proper types however we are forcing the requirement for ZMMWORD type reference.. that means that:

   vmovdqa zmm1, ZMMWORD PTR ZMM_BITS30  ;this will work

   ;vmovdqa zmm1, YMMWORD PTR ZMM_BITS30 ;this will fail (Cannot move YMMWORD type to ZMM(n)

   vmovdqa ymm1, YMMWORD PTR YMM_BITS30  ;this will work

   ;vmovdqa zmm1, ZMM_BITS30             ;this will fail  (no ZMMWORD type, source data is DD type)

   ;vmovdqa ymm1, YMM_BITS30             ;this will work

Regards,

John

From: gwoltman [mailto:notifications@github.com] Sent: 18 September 2016 03:52 PM To: Terraspace/HJWasm HJWasm@noreply.github.com Cc: John Hankinson john@terraspace.co.uk; Comment comment@noreply.github.com Subject: Re: [Terraspace/HJWasm] Minor AVX512 issue (#32)

Hi,

Thanks for looking into these issues.

"abc" was really ZMM_BITS30 declared as follows:

ZMM_BITS30 DD 3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0

Best regards, George Woltman

On Sun, Sep 18, 2016 at 5:59 AM, John Hankinson notifications@github.com wrote:

Can you send me the data declaration for abc ?

I’ve tried a few different combinations of this and it appears to work for me.

From: gwoltman [mailto:notifications@github.com] Sent: 18 September 2016 04:04 AM To: Terraspace/HJWasm HJWasm@noreply.github.com Subject: [Terraspace/HJWasm] Minor AVX512 issue (#32)

vmovdqa zmm1, ZMMWORD PTR abc

does not assemble, but

vmovdqa zmm1, YMMWORD PTR abc

does

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ Terraspace/HJWasm/issues/32 , or mute the thread https://github.com/ notifications/unsubscribe-auth/AQGQVC4Pld4WB5oBol4AUjpHGvkflc 7uks5qrKoYgaJpZM4J_yFi .

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Terraspace/HJWasm/issues/32#issuecomment-247838591, or mute the thread https://github.com/notifications/unsubscribe-auth/AVPFMwU7daXWavrNY-b5J6Fl6oEp3jOVks5qrQuEgaJpZM4J_yFi .

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Terraspace/HJWasm/issues/32#issuecomment-247852086 , or mute the thread https://github.com/notifications/unsubscribe-auth/AQGQVHAIXkYy0aSdIoWnqDYw5GDa1UQTks5qrU_7gaJpZM4J_yFi .

john-terraspace commented 8 years ago

Please download the latest package from the site and re-test. This latest build of 2.15 r2 should solve the AVX2, ZMM encoding issues you have had (plus a few more we found!).

Regards,

John

From: gwoltman [mailto:notifications@github.com] Sent: 18 September 2016 03:52 PM To: Terraspace/HJWasm HJWasm@noreply.github.com Cc: John Hankinson john@terraspace.co.uk; Comment comment@noreply.github.com Subject: Re: [Terraspace/HJWasm] Minor AVX512 issue (#32)

Hi,

Thanks for looking into these issues.

"abc" was really ZMM_BITS30 declared as follows:

ZMM_BITS30 DD 3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0,3FFFFFFFh,0

Best regards, George Woltman

On Sun, Sep 18, 2016 at 5:59 AM, John Hankinson notifications@github.com wrote:

Can you send me the data declaration for abc ?

I’ve tried a few different combinations of this and it appears to work for me.

From: gwoltman [mailto:notifications@github.com] Sent: 18 September 2016 04:04 AM To: Terraspace/HJWasm HJWasm@noreply.github.com Subject: [Terraspace/HJWasm] Minor AVX512 issue (#32)

vmovdqa zmm1, ZMMWORD PTR abc

does not assemble, but

vmovdqa zmm1, YMMWORD PTR abc

does

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ Terraspace/HJWasm/issues/32 , or mute the thread https://github.com/ notifications/unsubscribe-auth/AQGQVC4Pld4WB5oBol4AUjpHGvkflc 7uks5qrKoYgaJpZM4J_yFi .

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Terraspace/HJWasm/issues/32#issuecomment-247838591, or mute the thread https://github.com/notifications/unsubscribe-auth/AVPFMwU7daXWavrNY-b5J6Fl6oEp3jOVks5qrQuEgaJpZM4J_yFi .

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Terraspace/HJWasm/issues/32#issuecomment-247852086 , or mute the thread https://github.com/notifications/unsubscribe-auth/AQGQVHAIXkYy0aSdIoWnqDYw5GDa1UQTks5qrU_7gaJpZM4J_yFi .