Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Test27 for Android #1461

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

KreitinnSoftware
Copy link
Contributor

No description provided.

@ptitSeb
Copy link
Owner

ptitSeb commented Apr 23, 2024

can you re-enable test27 alse?

@ptitSeb
Copy link
Owner

ptitSeb commented Apr 23, 2024

Mmmm, test is failling.

@KreitinnSoftware
Copy link
Contributor Author

KreitinnSoftware commented May 10, 2024

on my local tests, without dynarec, test27 (sse4.2 test) is working proper, and with dynarec (i tried with safeflags=2 and strongmem=3) not working proper
box64_android_test27_output_with_dynarec.log
box64_android_test27_output_without_dynarec.log

Using diff get this

[pablo@pablo-ms7817 tests]$ diff box64_android_test27_output_with_dynarec.log ref27_android.txt
207c207
< _mm_cmpestri("This", 4, "This is a string", 16, 0x8) flags: a:0 s:0 z:0 c:1 o:1

_mm_cmpestri("This", 4, "This is a string", 16, 0x8) flags: a:0 s:1 z:0 c:1 o:1
213c213
< _mm_cmpestri("This", 4, "This is a string", 16, 0x9) flags: a:0 s:0 z:0 c:1 o:1


_mm_cmpestri("This", 4, "This is a string", 16, 0x9) flags: a:0 s:1 z:0 c:1 o:1
231c231
< _mm_cmpestri("is", 2, "This is a string", 16, 0x8) flags: a:1 s:0 z:0 c:0 o:0


_mm_cmpestri("is", 2, "This is a string", 16, 0x8) flags: a:1 s:1 z:0 c:0 o:0
237c237
< _mm_cmpestri("is", 2, "This is a string", 16, 0x9) flags: a:1 s:0 z:0 c:0 o:0


_mm_cmpestri("is", 2, "This is a string", 16, 0x9) flags: a:1 s:1 z:0 c:0 o:0
267c267
< _mm_cmpestri("This is a string", 16, "", 0, 0x8) flags: a:0 s:1 z:1 c:0 o:0


_mm_cmpestri("This is a string", 16, "", 0, 0x8) flags: a:0 s:0 z:1 c:0 o:0
273c273
< _mm_cmpestri("This is a string", 16, "", 0, 0x9) flags: a:0 s:1 z:1 c:0 o:0


_mm_cmpestri("This is a string", 16, "", 0, 0x9) flags: a:0 s:0 z:1 c:0 o:0
279c279
< _mm_cmpestri("", 0, "This is a string", 16, 0x8) flags: a:1 s:0 z:0 c:0 o:0


_mm_cmpestri("", 0, "This is a string", 16, 0x8) flags: a:1 s:1 z:0 c:0 o:0
285c285
< _mm_cmpestri("", 0, "This is a string", 16, 0x9) flags: a:1 s:0 z:0 c:0 o:0


_mm_cmpestri("", 0, "This is a string", 16, 0x9) flags: a:1 s:1 z:0 c:0 o:0

@ksco
Copy link
Collaborator

ksco commented May 10, 2024

Try to use BOX64_DYNAREC_TEST=1 box64 ./test27 on your local machine to see if there is any DIFF xxx logs.

@KreitinnSoftware
Copy link
Contributor Author

Try to use BOX64_DYNAREC_TEST=1 box64 ./test27 on your local machine to see if there is any DIFF xxx logs.

I will test when I get home

@KreitinnSoftware
Copy link
Contributor Author

Try to use BOX64_DYNAREC_TEST=1 box64 ./test27 on your local machine to see if there is any DIFF xxx logs.

box64-android-dynarec-test.log

@ksco
Copy link
Collaborator

ksco commented May 10, 2024

So, the diffs are what you need to look into and fix..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants