Skip to content

Conversation

runlevel5
Copy link
Contributor

@runlevel5 runlevel5 commented Nov 13, 2022

This would add a build config for PowerPC 64 Little Endian architecture with gcc

It's part of my effort to get LWJGL working for game development on the OpenPOWER platform.

Many thanks

How to build?

# make sure you build bx/genie binary on your ppc64le first because the bundled genie is for x86_64
./tools/bin/linux/genie  --gcc=linux-ppc64le-gcc gmake

make  -R -C .build/projects/gmake-linux-ppc64le-gcc/ config=debug64

@bkaradzic bkaradzic closed this Nov 13, 2022
@runlevel5
Copy link
Contributor Author

@bkaradzic I hope you are well and sound. I would like to ask the PR is rejected please.

@bkaradzic
Copy link
Owner

image

You provided no description about it. So I assumed I don't have to provide reason either...

@runlevel5
Copy link
Contributor Author

@bkaradzic my bad. Thanks for the feedback.

@runlevel5
Copy link
Contributor Author

@bkaradzic I have just added description for the PR. If possible, would you be able to re-open the PR? Thanks for the consideration

@bkaradzic
Copy link
Owner

Which OpenPPC you're using for testing?
Do you need big endian?
Is this the only change that's needed for bgfx to work on Linux PPC or there will be more changes?

@runlevel5
Copy link
Contributor Author

Which OpenPPC you're using for testing?

Mainly LE variant but I do believe some devs might find BE useful.


Is this the only change that's needed for bgfx to work on Linux PPC or there will be more changes?

This is the only change.

@bkaradzic
Copy link
Owner

Mainly LE variant but I do believe some devs might find BE useful.

I do not plan to support BE at all.

This is the only change.

What's the difference if you just compile it with linux-gcc instead?

@runlevel5
Copy link
Contributor Author

runlevel5 commented Nov 14, 2022

@bkaradzic the linux-gcc profile has build option for SSE which is not available in PowerISA

make  -R -C .build/projects/gmake-linux config=debug64
make: Entering directory '/home/tle/Work/bx/.build/projects/gmake-linux'
==== Building bx (debug64) ====
Creating ../../linux64_gcc/obj/x64/Debug/bx
Creating ../../linux64_gcc/obj/x64/Debug/bx/src
Creating ../../linux64_gcc/bin
allocator.cpp
g++: error: unrecognized command-line option ‘-mfpmath=sse’
g++: error: unrecognized command-line option ‘-msse2’
make[1]: *** [bx.make:441: ../../linux64_gcc/obj/x64/Debug/bx/src/allocator.o] Error 1
make: *** [Makefile:17: bx] Error 2
make: Leaving directory '/home/tle/Work/bx/.build/projects/gmake-linux'

P/S: I compiled the genie locally so I could run on ppc64le linux

@runlevel5
Copy link
Contributor Author

runlevel5 commented Nov 14, 2022

I do not plan to support BE at all.

👍 I added a new commit to ensure that

Ref: runlevel5@f63daba

@runlevel5
Copy link
Contributor Author

Is this the only change that's needed for bgfx to work on Linux PPC or there will be more changes?

It should be enough to unblock me with LWJGL Pull Request LWJGL-CI/bgfx#3

@runlevel5 runlevel5 changed the title Support PPC Linux Support PPC64 LE Linux Nov 14, 2022
@bkaradzic bkaradzic reopened this Nov 14, 2022
@bkaradzic bkaradzic merged commit 8d276a1 into bkaradzic:master Nov 14, 2022
@runlevel5
Copy link
Contributor Author

@bkaradzic great thanks for reviewing this PR.

I would like to ask if you plan to bundle ppc64le version of genie in ./tools/bin/linux/? If the answer is yes, I could make a pull request

Kind regards

@runlevel5 runlevel5 deleted the ppc branch November 14, 2022 06:16
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.

2 participants