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

Add new CPU architecture #49

Closed
wangchenguang123 opened this issue Dec 11, 2023 · 4 comments
Closed

Add new CPU architecture #49

wangchenguang123 opened this issue Dec 11, 2023 · 4 comments

Comments

@wangchenguang123
Copy link
Contributor

When I want to add a new form of CPU architecture to this project, how can I test its success

@SimonIT
Copy link
Member

SimonIT commented Dec 11, 2023

If you run ./gradlew build (or just ./gradlew test) on a computer with the new architecture, it will do all the jnigen magic and run it afterwords. If it is successful, something like this:

boolean: true
byte: 1
char: �
short: 3
int: 4
long: %
float: 6.000000
double: 0
byteBuffer: 8
bool[0]: false
char[0]:
short[0]: 10
int[0]: 11
long[0]: %
float[0]: 13.000000
double[0]: 14.000000
string: Hurray fuck this nuts

will appear in the log

@wangchenguang123
Copy link
Contributor Author

If you run ./gradlew build (or just ./gradlew test) on a computer with the new architecture, it will do all the jnigen magic and run it afterwords. If it is successful, something like this:

boolean: true
byte: 1
char: �
short: 3
int: 4
long: %
float: 6.000000
double: 0
byteBuffer: 8
bool[0]: false
char[0]:
short[0]: 10
int[0]: 11
long[0]: %
float[0]: 13.000000
double[0]: 14.000000
string: Hurray fuck this nuts

will appear in the log

ok thank you for your reply when i try to change some for new arch LoongArch like this。but i faifed do ./gradlew test
i will continue try to find error about this
图片
图片

@SimonIT
Copy link
Member

SimonIT commented Dec 11, 2023

You can find a more detailed exception in gdx-jnigen/build/reports/tests/test/classes/com.badlogic.gdx.jnigen.JniGenTest.html. Probably there are some tools missing

@Berstanio
Copy link
Contributor

#54 has been merged

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

No branches or pull requests

3 participants