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

Allow compiler optimisations with -fno-delete-null-pointer-checks #131

Merged

Conversation

chewi
Copy link
Contributor

@chewi chewi commented Aug 14, 2019

If at least -O1 is given then the client crashes on startup without
disabling this optimisation. It is supported at least since GCC 3.4.

This was a tricky one to find because -O0 works and -O1 doesn't but
the GCC manual says this optimisation is only enabled at -O2. Turns
out it is also enabled by default on most targets.

It may be possible to narrow it down to a specific subproject or
source file but applying it globally seems safest.

If at least -O1 is given then the client crashes on startup without
disabling this optimisation. It is supported at least since GCC 3.4.

This was a tricky one to find because -O0 works and -O1 doesn't but
the GCC manual says this optimisation is only enabled at -O2. Turns
out it is also enabled by default on most targets.

It may be possible to narrow it down to a specific subproject or
source file but applying it globally seems safest.
@Titaniumtown
Copy link

I built mcpelauncher with this cxx flag, it is way faster! Hope it gets merged!

@MCMrARM MCMrARM merged commit a94d2ef into minecraft-linux:master Sep 13, 2019
@RianFC RianFC mentioned this pull request Nov 1, 2019
@winter0729 winter0729 mentioned this pull request Nov 12, 2019
@zvillager10 zvillager10 mentioned this pull request Nov 18, 2019
@oakboat oakboat mentioned this pull request Nov 30, 2019
@BRENTGAMEZ BRENTGAMEZ mentioned this pull request Dec 23, 2019
@Murtadha27827 Murtadha27827 mentioned this pull request Jul 13, 2020
@chewi chewi deleted the fno-delete-null-pointer-checks branch October 19, 2021 21:35
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.

3 participants