Skip to content

Commit 0c25fa4

Browse files
author
Jonah Williams
authored
[Impeller] re-enable Adreno 630 (#161287)
Part of flutter/flutter#161209
1 parent d8c4fc0 commit 0c25fa4

File tree

2 files changed

+5
-11
lines changed

2 files changed

+5
-11
lines changed

engine/src/flutter/impeller/renderer/backend/vulkan/driver_info_vk.cc

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -331,16 +331,10 @@ bool DriverInfoVK::IsEmulator() const {
331331
bool DriverInfoVK::IsKnownBadDriver() const {
332332
if (adreno_gpu_.has_value()) {
333333
AdrenoGPU adreno = adreno_gpu_.value();
334-
// See:
335-
// https://github.com/flutter/flutter/issues/154103
336-
//
337-
// Reports "VK_INCOMPLETE" when compiling certain entity shader with
338-
// vkCreateGraphicsPipelines, which is not a valid return status.
339-
// See https://github.com/flutter/flutter/issues/155185 .
340-
//
341-
// https://github.com/flutter/flutter/issues/155185
342-
// Unknown crashes but device is not easily acquirable.
343-
if (adreno <= AdrenoGPU::kAdreno630) {
334+
// 630 is the lowest version I've tested on the still works.
335+
// I suspect earlier 600s should work but this is waiting on
336+
// more devices for testing.
337+
if (adreno < AdrenoGPU::kAdreno630) {
344338
return true;
345339
}
346340
}

engine/src/flutter/impeller/renderer/backend/vulkan/driver_info_vk_unittests.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,6 @@ TEST(DriverInfoVKTest, DriverParsingAdreno) {
163163
}
164164

165165
TEST(DriverInfoVKTest, DisabledDevices) {
166-
EXPECT_TRUE(IsBadVersionTest("Adreno (TM) 630"));
167166
EXPECT_TRUE(IsBadVersionTest("Adreno (TM) 620"));
168167
EXPECT_TRUE(IsBadVersionTest("Adreno (TM) 610"));
169168
EXPECT_TRUE(IsBadVersionTest("Adreno (TM) 530"));
@@ -174,6 +173,7 @@ TEST(DriverInfoVKTest, DisabledDevices) {
174173
EXPECT_TRUE(IsBadVersionTest("Adreno (TM) 505"));
175174
EXPECT_TRUE(IsBadVersionTest("Adreno (TM) 504"));
176175

176+
EXPECT_FALSE(IsBadVersionTest("Adreno (TM) 630"));
177177
EXPECT_FALSE(IsBadVersionTest("Adreno (TM) 640"));
178178
EXPECT_FALSE(IsBadVersionTest("Adreno (TM) 650"));
179179
}

0 commit comments

Comments
 (0)