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 option to remove Intel-specific JVM arg for compatibility #5477

Open
1 task done
Arminias opened this issue Jul 22, 2024 · 0 comments
Open
1 task done

Add option to remove Intel-specific JVM arg for compatibility #5477

Arminias opened this issue Jul 22, 2024 · 0 comments
Labels
suggestion Feature suggestions that have not been accepted yet.

Comments

@Arminias
Copy link

Role

I play modded MC and am one of the maintainers of BTW

Suggestion

Add option to remove Intel-specific JVM arg for compatibility (e.g. in the instance's Workaround tab or similar)

Benefit

This will make people on old Intel iGPUs be able to - if they wish so - use other Java than 8 u51. Especially required on old MC versions when needing to use more modern Java.

This suggestion is unique

  • I have searched the issue tracker and did not find an issue describing my suggestion, especially not one that has been rejected.

You may use the editor below to elaborate further.

The arg in question is MojangTricksIntelDriversForPerformance_javaw.exe_minecraft.exe.heapdump. Seen here:

args << QString("-XX:HeapDumpPath=MojangTricksIntelDriversForPerformance_javaw.exe_"

This always gets added on Windows and can't be removed by the user right now.
Removing it can cause varying performance degradation - therefore I suggest adding it in the Workaround tab - though not always significant.
This will make affected users be able to use other Java versions than 8 u51 when enabled and make them able to launch old MC versions on modern Java without any problems.
(Regarding 8 u51, see this MultiMC wiki entry: https://github.com/MultiMC/Launcher/wiki/Unsupported-Intel-GPUs)

I tested this on a custom build of MultiMC I did myself that removed this arg and other versions than 8 u51 started working without any issues.

@phit phit added the suggestion Feature suggestions that have not been accepted yet. label Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
suggestion Feature suggestions that have not been accepted yet.
Projects
None yet
Development

No branches or pull requests

2 participants