search

Terraspace / UASM

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

Another AVX-512 issue #39

Closed gwoltman closed 7 years ago

gwoltman commented 7 years ago

vfnmadd231pd zmm27, zmm24, zmm23

assembles as

    ` 62 02 BD 40: BC. DF`

which disssembles to:

    `vfnmadd231pd zmm27, zmm24, zmm31 `
gwoltman commented 7 years ago

and an AVX2 issue too:

vpsrlq ymm9, ymm7, [r11]

assembles as:

vpsrlq ymm9, ymm7, xmmword ptr [rbx] ; 3BF8 _ C5 45: D3. 0B

john-terraspace commented 7 years ago

All these issues are fixed, new version/package on the site dated 14th November.

Cheers

John

From: gwoltman [mailto:notifications@github.com] Sent: 14 November 2016 04:52 AM To: Terraspace/HJWasm HJWasm@noreply.github.com Subject: Re: [Terraspace/HJWasm] Another AVX-512 issue (#39)

and an AVX2 issue too:

vpsrlq ymm9, ymm7, [r11]

assembles as:

vpsrlq ymm9, ymm7, xmmword ptr [rbx] ; 3BF8 _ C5 45: D3. 0B

— 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/39#issuecomment-260248105 , or mute the thread https://github.com/notifications/unsubscribe-auth/AQGQVApZR8x02WV_8Ts2ReuV3UYLvl0Zks5q9-jagaJpZM4Kw5vy .

gwoltman commented 7 years ago

A little bit closer each day...

vpsrlq ymm9, ymm9, 30

assembles as

vpsrlq ymm9, ymm1, 30

(vpsllq and vpsrad also)

john-terraspace commented 7 years ago

Goodness.. I hope we’re nearly at the end of them!

I really think once this is all 100% we need to put some complete regression tests in, against known binary outputs.

From: gwoltman [mailto:notifications@github.com] Sent: 14 November 2016 04:30 PM To: Terraspace/HJWasm HJWasm@noreply.github.com Cc: John Hankinson john@terraspace.co.uk; Comment comment@noreply.github.com Subject: Re: [Terraspace/HJWasm] Another AVX-512 issue (#39)

A little bit closer each day...

vpsrlq ymm9, ymm9, 30

assembles as

vpsrlq ymm9, ymm1, 30

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

john-terraspace commented 7 years ago

Affects the whole rotate/shift group:

vpsrlq ymm9, ymm9, 30

000000013FE71000 C5 35 73 D1 1E vpsrlq ymm9,ymm1,1Eh 

69:       vpsllq ymm9, ymm9, 30

000000013FE71005 C5 35 73 F1 1E vpsllq ymm9,ymm1,1Eh 

70:       vpslldq ymm9,ymm9, 30

000000013FE7100A C5 B5 73 F9 1E vpslldq ymm9,ymm1,1Eh 

71:       vpsllw ymm8,ymm9,30

000000013FE7100F C5 3D 71 F1 1E vpsllw ymm8,ymm1,1Eh 

72:       vpslld ymm9, ymm9, 30

000000013FE71014 C5 35 72 F1 1E vpslld ymm9,ymm1,1Eh 

73:       vpsraw ymm9, ymm9, 30

000000013FE71019 C5 35 71 E1 1E vpsraw ymm9,ymm1,1Eh 

74:       vpsrad ymm9,ymm9, 30

000000013FE7101E C5 35 72 E1 1E vpsrad ymm9,ymm1,1Eh 

75:       vpsrldq ymm9, ymm9, 30

000000013FE71023 C5 B5 73 D9 1E vpsrldq ymm9,ymm1,1Eh

From: gwoltman [mailto:notifications@github.com] Sent: 14 November 2016 04:30 PM To: Terraspace/HJWasm HJWasm@noreply.github.com Cc: John Hankinson john@terraspace.co.uk; Comment comment@noreply.github.com Subject: Re: [Terraspace/HJWasm] Another AVX-512 issue (#39)

A little bit closer each day...

vpsrlq ymm9, ymm9, 30

assembles as

vpsrlq ymm9, ymm1, 30

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

john-terraspace commented 7 years ago

Updated packages on the site. All shift/rotates fixed as per list in last note.

From: gwoltman [mailto:notifications@github.com] Sent: 14 November 2016 04:30 PM To: Terraspace/HJWasm HJWasm@noreply.github.com Cc: John Hankinson john@terraspace.co.uk; Comment comment@noreply.github.com Subject: Re: [Terraspace/HJWasm] Another AVX-512 issue (#39)

A little bit closer each day...

vpsrlq ymm9, ymm9, 30

assembles as

vpsrlq ymm9, ymm1, 30

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

john-terraspace commented 7 years ago

We’ve also found an issue with the evex encoded versions:

   vpsrlq ymm1, ymm20, 30                   

   vpsllq ymm1, ymm20, 30                   

   vpslldq ymm1,ymm20, 30

(Register > 15) Fixing these now.

From: gwoltman [mailto:notifications@github.com] Sent: 14 November 2016 04:30 PM To: Terraspace/HJWasm HJWasm@noreply.github.com Cc: John Hankinson john@terraspace.co.uk; Comment comment@noreply.github.com Subject: Re: [Terraspace/HJWasm] Another AVX-512 issue (#39)

A little bit closer each day...

vpsrlq ymm9, ymm9, 30

assembles as

vpsrlq ymm9, ymm1, 30

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

gwoltman commented 7 years ago

Not quite:

vpsrlq ymm9, ymm7, 3

now assembles as:

vpsrlq ymm9, ymm15, 3 ; 3B6F _ C4 C1 B5: 73. D7, 03

habran commented 7 years ago

Still working Will be ready tonight

On Tue, Nov 15, 2016 at 6:17 AM, gwoltman notifications@github.com wrote:

Not quite:

vpsrlq ymm9, ymm7, 3

now assembles as:

vpsrlq ymm9, ymm15, 3 ; 3B6F _ C4 C1 B5: 73. D7, 03

— 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/39#issuecomment-260441912, or mute the thread https://github.com/notifications/unsubscribe-auth/AQH-YL3Fk4EDAWJKpNlPny17XaTvnyOGks5q-LrrgaJpZM4Kw5vy .

gwoltman commented 7 years ago

All fixed. Thanks!