mojira.dev
MC-239582

net.minecraft.util.math.MathHelper

I apologize immediately for possible mistakes, I am not a native English speaker.
It will be about net.minecraft.util.math.MathHelper methods such as (sign, floor, ceil, max/min and others (not including optimized sin/cos functions) since minecraft 1.17, which starts working on java 16+ java.lang.Math more and more methods have IntrinsicCandidate. In the code of minecraft itself many inline methods of max/min methods for example, which can be safely changed to optimized java.lang.Math.max/min methods The lion's share of methods from net.minecraft.util.math.MathHelper doesn't make sense and will be slower than java.lang.Math (depends on the platform), I suggest cutting out the entire legacy code.
I understand I illegally looked at the minecraft code, but I hope it will help improve the performance of the game with minimal effort, because these methods are called often enough

Comments 7

⚠️ Please do not mark Unreleased Versions as affected. You don't have access to them yet.

-- I am a bot. This action was performed automatically! If you think it was incorrect, please notify us on Discord or Reddit

⚠️ Please do not mark Unreleased Versions as affected. You don't have access to them yet.

-- I am a bot. This action was performed automatically! If you think it was incorrect, please notify us on Discord or Reddit

Thank you for your report!
However, this issue is Invalid.

You have posted a feature request or a suggestion. This site is for bug reports only.
For suggestions, please visit Minecraft Suggestions on Reddit or visit the Feedback website.

Quick Links:
📓 Bug Tracker Guidelines – 💬 Community Support – 📧 Mojang Support
📓 Project Summary – ✍️ Feedback and Suggestions – 📖 Game Wiki

Are you sure this is considered a feature request? Other code-optimization reports have been accepted before.

"I understand I illegally looked at the minecraft code"

You're free to decompile the Java Edition of Minecraft, nothing is wrong with that, don't worry about it.

This class is called net.minecraft.util.Mth in Mojang's mappings.

This optimization is done in OptiFine if I'm not mistaken. I don't know if this information is relevant.

Zelva Lea

(Unassigned)

Plausible

Performance

1.17.1, 21w42a

Retrieved