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

neo/CMakeLists.txt: added use of O3 instead of O2 for e2k arch #650

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

r-a-sattarov
Copy link
Contributor

Using O3 option on E2K mcst-lcc compiler approximately equal to O2 at X86/ARM gcc.

O3 on E2K mcst-lcc approximately equal to O2 at X86/ARM gcc
@EntityFX
Copy link

EntityFX commented Feb 7, 2025

Looks good

@DanielGibson
Copy link
Member

if you say so.. it's not like I have to debug that shit if weird issues happen due to aggressive optimization (because I don't have such a CPU) ¯\(ツ)

@DanielGibson DanielGibson merged commit a18b9e6 into dhewm:master Feb 7, 2025
4 checks passed
@r-a-sattarov
Copy link
Contributor Author

The O3 optimization level on MCST-LCC is almost equal to O2 on GCC. So everything will be fine.
And O4 on MCST-LCC (yes, there is one) is approximately equal to O3 GCC.

Ref: http://mcst.ru/doc/elbrus_prog/html/chapter1.html#lcc

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

Successfully merging this pull request may close these issues.

3 participants