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

Gfxbench Manhattan3.0 optimization #107

Open
wants to merge 1 commit into
base: celadon/s/mr0/apollo
Choose a base branch
from

Conversation

huangrui666
Copy link

@huangrui666 huangrui666 commented Jun 17, 2024

Eliminate draw call for Manhattan robot shadow when conditions apply. The performance of Gfxbench Manhattan3.0 offsreen can be improved 2.4%

Tracked-On: OAM-122053

@huangrui666 huangrui666 force-pushed the mht_opt branch 2 times, most recently from 1db5e70 to 7f8c3fb Compare July 8, 2024 02:42
Copy link

@feijiang1 feijiang1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Eliminate draw call for Manhattan robot shadow when conditions
apply. Mark the shadow shader. If the draw call uses the shadow
shader and the last operation of its depth texture resource is
clear, the draw call can be skiped.
The performance of Gfxbench 1080pManhattan3.0 offscreen can
be improved 1.5%.

Test Done:
1. Boot up and adb reboot
2. Gfxbench

Tracked-On: OAM-122053
Signed-off-by: Huang Rui <[email protected]>
@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci added the Engineering Build Successful Engineering Build Successful label Jul 17, 2024
@sysopenci sysopenci added the Stale Stale label for inactive open prs label Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Developer Approved Engineering Build Successful Engineering Build Successful Stale Stale label for inactive open prs Valid commit message
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants