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

URP Performance Improvements #7653

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

smitdylan2001
Copy link
Contributor

Some simple performance improvements. Would be good to take these into new commits as well and could be backported
Since nothing changed for the end user I did not make changelog edits. Is that necessary?

Improved CPU performance with:

  • Re ordering vector calculations
  • Removing indirect gameObject call
  • Simplify LINQ comparison
  • Using native lenght or count instead of LINQ
  • Simplify string comparisons
  • Merge if/else return into 1 return statement

Purpose of this PR

Why is this PR needed, what hard problem is it solving/fixing?
Improve CPU performance with minor fixes


Testing status

Describe what manual/automated tests were performed for this PR
No major changes made to code


Comments to reviewers

Notes for the reviewers you have assigned.
Do I need to add changelog changes?
Can this be backported?

Some simple performance improvements. Would be good to take these into new commits as well and could be backported

Improved CPU performance with:

- Re ordering vector calculations
- Removing indirect gameObject call
- Simplify LINQ comparison
- Using native lenght or count instead of LINQ
- Simplify string comparisons
- Merge if/else return into 1 return statement
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants