Remove storage bit of render target in Mobile renderer

beicause · beicause · commit 6970d355c33b · 2025-09-28T00:38:33.000+08:00
diff --git a/servers/rendering/renderer_rd/storage_rd/texture_storage.cpp b/servers/rendering/renderer_rd/storage_rd/texture_storage.cpp
@@ -4596,7 +4596,10 @@ uint32_t TextureStorage::render_target_get_color_usage_bits(bool p_msaa) {
 	if (p_msaa) {
 		return RD::TEXTURE_USAGE_COLOR_ATTACHMENT_BIT;
 	} else {
-		// FIXME: Storage bit should only be requested when FSR is required.
-		return RD::TEXTURE_USAGE_SAMPLING_BIT | RD::TEXTURE_USAGE_COLOR_ATTACHMENT_BIT | RD::TEXTURE_USAGE_CAN_COPY_FROM_BIT | RD::TEXTURE_USAGE_STORAGE_BIT;
+		uint32_t usage_bits = RD::TEXTURE_USAGE_SAMPLING_BIT | RD::TEXTURE_USAGE_COLOR_ATTACHMENT_BIT | RD::TEXTURE_USAGE_CAN_COPY_FROM_BIT;
+		if (RendererSceneRenderRD::get_singleton()->_render_buffers_can_be_storage()) {
+			usage_bits |= RD::TEXTURE_USAGE_STORAGE_BIT;
+		}
+		return usage_bits;
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -4596,7 +4596,10 @@ uint32_t TextureStorage::render_target_get_color_usage_bits(bool p_msaa) {`
`4596`	`4596`	`if (p_msaa) {`
`4597`	`4597`	`return RD::TEXTURE_USAGE_COLOR_ATTACHMENT_BIT;`
`4598`	`4598`	`} else {`
`4599`		`- // FIXME: Storage bit should only be requested when FSR is required.`
`4600`		`- return RD::TEXTURE_USAGE_SAMPLING_BIT \| RD::TEXTURE_USAGE_COLOR_ATTACHMENT_BIT \| RD::TEXTURE_USAGE_CAN_COPY_FROM_BIT \| RD::TEXTURE_USAGE_STORAGE_BIT;`
	`4599`	`+ uint32_t usage_bits = RD::TEXTURE_USAGE_SAMPLING_BIT \| RD::TEXTURE_USAGE_COLOR_ATTACHMENT_BIT \| RD::TEXTURE_USAGE_CAN_COPY_FROM_BIT;`
	`4600`	`+ if (RendererSceneRenderRD::get_singleton()->_render_buffers_can_be_storage()) {`
	`4601`	`+ usage_bits \|= RD::TEXTURE_USAGE_STORAGE_BIT;`
	`4602`	`+ }`
	`4603`	`+ return usage_bits;`
`4601`	`4604`	`}`
`4602`	`4605`	`}`