disable gpu preprocessing on android with Adreno 730 GPU and earilier #14176
Conversation
|
This limits too much because 740 is proven to work #13323 (comment), and 750 works for me. |
Litttlefish
changed the title
I've changed it to only disble on 6xx and 730 now |
|
There are also 702, 710, 720, 725 that I think are safe to assume don't work either because they are older. |
|
So I would check that the second number is 3 or less. |
Litttlefish
changed the title
My assumption is GPU earlier than 730(probably, since there Adreno 740 is used for 8Gen2 series) doesn't support it, so I used this for now But Adreno number naming is not that simple, like 720 is used in 7Gen3, which probably should support it(newer), while 725 used in 7Gen2 is the same to 730 with lower frequency🤔needs more testing to confirm |
|
Yeah looks like 732 and 735 are very new as well |
Anyways I excluded 720, since it probably supports gpu preprocessing, I'll let it for others to test |
alice-i-cecile
added
A-Rendering
# Objective The Android example on Adreno 642L currently crashes on startup. Previous PRs #14176 and #13323 have adressed this specific crash occurring on some Adreno GPUs, that fix works as it should but isn't applied when to the GPU name contains a suffix like in the case of `642L`. ## Solution - Amending the logic to filter out any parts of the GPU name not containing digits thus enabling the fix on `642L`. ## Testing - Ran the Android example on a Nothing Phone 1. Before this change it crashed, after it works as intended. --------- Co-authored-by: Sam Pettersson <[email protected]>
Objective
Fix #14146
Solution
Expansion of #13323 , excluded Adreno 730 and earlier.
Testing
Tested on android device(Adreno 730) that used to crash