search

chipsalliance / VeeR-ISS

Apache License 2.0
114 stars 33 forks source link

Issue with rmm mode (softfloat) in fcvt.s.w and fcvt.s.wu instructions #10

Closed UrvishkumarPatel closed 3 years ago

UrvishkumarPatel commented 3 years ago

I compiled whisper with "make SOFT_FLOAT=1". I ran the following commands

whisper -i --isa imafdc
whisper> poke c mstatus 0x80007800
whisper> poke r x22 0xfee0001b
whisper> poke m 0 0xd00b4c53
whisper> step
#1 0 00000000 d00b4c53 f 18 ffffffffcb8ffff2  fcvt.s.w f24, x22, rmm  +
#1 0 00000000 d00b4c53 c 0003       00000001  fcvt.s.w f24, x22, rmm
whisper> peek f f24
0xffffffffcb8ffff2

The expected result is 0xffffffffcb8ffff3

jrahmeh commented 3 years ago

Hi Urvish,

I am getting the expected results. Please make sure you have the latest from GitHub.

Joe

On Wed, Aug 18, 2021, 3:27 AM Urvish Patel @.***> wrote:

I compiled whisper with "make SOFT_FLOAT=1". I ran the following commands

whisper -i --isa imafdc whisper> poke c mstatus 0x80007800 whisper> poke r x22 0xfee0001b whisper> poke m 0 0xd00b4c53 whisper> step

1 0 00000000 d00b4c53 f 18 ffffffffcb8ffff2 fcvt.s.w f24, x22, rmm +

1 0 00000000 d00b4c53 c 0003 00000001 fcvt.s.w f24, x22, rmm

whisper> peek f f24 0xffffffffcb8ffff2

The expected result is 0xffffffffcb8ffff3

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/chipsalliance/SweRV-ISS/issues/10, or unsubscribe https://github.com/notifications/unsubscribe-auth/AASHQX7SUYHHDYMHSADFZ53T5NVHFANCNFSM5CLQRNNQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

jrahmeh commented 3 years ago

I updated the makefile. Please try again with latest code from GitHub.

On Wed, Aug 18, 2021, 3:27 AM Urvish Patel @.***> wrote:

I compiled whisper with "make SOFT_FLOAT=1". I ran the following commands

whisper -i --isa imafdc whisper> poke c mstatus 0x80007800 whisper> poke r x22 0xfee0001b whisper> poke m 0 0xd00b4c53 whisper> step

1 0 00000000 d00b4c53 f 18 ffffffffcb8ffff2 fcvt.s.w f24, x22, rmm +

1 0 00000000 d00b4c53 c 0003 00000001 fcvt.s.w f24, x22, rmm

whisper> peek f f24 0xffffffffcb8ffff2

The expected result is 0xffffffffcb8ffff3

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/chipsalliance/SweRV-ISS/issues/10, or unsubscribe https://github.com/notifications/unsubscribe-auth/AASHQX7SUYHHDYMHSADFZ53T5NVHFANCNFSM5CLQRNNQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

UrvishkumarPatel commented 3 years ago

Hi Joe, It is working now with the updated Makefile

Thanks, Urvish