xuanchi/XCEngine
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor(rendering): centralize stage graph output policies

Browse Source
This commit is contained in:
ssdfasd
2026-04-15 16:42:36 +08:00
parent c853f67c56
commit bf0b81d034
5 changed files with 48 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
engine/include/XCEngine/Rendering/Execution/CameraFrameStage.h
View File

@@ -45,6 +45,11 @@ enum class CameraFrameStageSharedSurfaceRole : uint8_t {
FinalComposited
};
enum class CameraFrameStagePublishedDepthAliasRole : uint8_t {
None,
MainDirectionalShadow
};
struct CameraFrameStageInfo {
CameraFrameStage stage = CameraFrameStage::MainScene;
const char* name = "";
@@ -198,6 +203,16 @@ GetCameraFrameStageSharedOutputSurfaceRole(CameraFrameStage stage) {
}
}
inline constexpr CameraFrameStagePublishedDepthAliasRole
GetCameraFrameStagePublishedDepthAliasRole(CameraFrameStage stage) {
switch (stage) {
case CameraFrameStage::ShadowCaster:
return CameraFrameStagePublishedDepthAliasRole::MainDirectionalShadow;
default:
return CameraFrameStagePublishedDepthAliasRole::None;
}
}
inline constexpr bool DoesCameraFrameStageFallbackUseColorOnlySurfaceAccess(
CameraFrameStage stage) {
return IsCameraFrameFullscreenSequenceStage(stage);

4
engine/src/Rendering/Execution/CameraFramePlan.cpp
View File

@@ -41,6 +41,10 @@ bool CameraFramePlan::UsesGraphManagedMainSceneColor() const {
}
bool CameraFramePlan::UsesGraphManagedOutputColor(CameraFrameStage stage) const {
if (stage == CameraFrameStage::MainScene) {
return UsesGraphManagedMainSceneColor();
}
if (const CameraFrameFullscreenStagePlan* fullscreenStagePlan =
GetFullscreenStagePlan(stage);
fullscreenStagePlan != nullptr) {

27
engine/src/Rendering/Execution/Internal/CameraFrameGraph/StageContract.cpp
View File

@@ -108,6 +108,23 @@ RenderGraphSurfaceAccessMode BuildCameraFrameStageFallbackSurfaceAccessMode(
: RenderGraphSurfaceAccessMode::ColorDepth;
}
void PublishCameraFrameStageGraphDepthAliasResource(
CameraFrameRenderGraphResources& frameResources,
const CameraFrameStageGraphBuildState& stageState,
const DirectionalShadowExecutionState& shadowState) {
switch (GetCameraFrameStagePublishedDepthAliasRole(stageState.stage)) {
case CameraFrameStagePublishedDepthAliasRole::MainDirectionalShadow:
if (shadowState.HasShadowSampling() &&
stageState.outputSurface.depthTexture.IsValid()) {
frameResources.mainDirectionalShadow =
stageState.outputSurface.depthTexture;
}
break;
default:
break;
}
}
} // namespace
CameraFrameRenderGraphSourceBinding BuildCameraFrameStageGraphSourceBinding(
@@ -168,12 +185,10 @@ void PublishCameraFrameStageGraphResources(
stageState.stage,
stageState.outputColor,
stageState.outputSurface.depthTexture);
if (stageState.stage == CameraFrameStage::ShadowCaster &&
context.shadowState.HasShadowSampling() &&
stageState.outputSurface.depthTexture.IsValid()) {
frameResources.mainDirectionalShadow = stageState.outputSurface.depthTexture;
}
PublishCameraFrameStageGraphDepthAliasResource(
frameResources,
stageState,
context.shadowState);
}
RenderSceneData BuildCameraFrameStandaloneStageSceneData(

7
engine/src/Rendering/Execution/Internal/CameraFrameGraph/StagePolicy.h
View File

@@ -16,12 +16,7 @@ struct CameraFrameRenderGraphSourceBinding {
inline bool UsesCameraFrameStageGraphManagedOutputColor(
const CameraFramePlan& plan,
CameraFrameStage stage) {
switch (stage) {
case CameraFrameStage::MainScene:
return plan.UsesGraphManagedMainSceneColor();
default:
return plan.UsesGraphManagedOutputColor(stage);
}
return plan.UsesGraphManagedOutputColor(stage);
}
inline CameraFrameColorSource ResolveCameraFrameStageGraphManagedColorSource(

7
tests/Rendering/unit/test_camera_frame_graph_stage_policy.cpp
View File

@@ -38,6 +38,12 @@ TEST(CameraFrameRenderGraphStagePolicy_Test, ReportsStageFullscreenAndTransition
EXPECT_EQ(
GetCameraFrameStageSharedOutputSurfaceRole(CameraFrameStage::PostProcess),
CameraFrameStageSharedSurfaceRole::None);
EXPECT_EQ(
GetCameraFrameStagePublishedDepthAliasRole(CameraFrameStage::ShadowCaster),
CameraFrameStagePublishedDepthAliasRole::MainDirectionalShadow);
EXPECT_EQ(
GetCameraFrameStagePublishedDepthAliasRole(CameraFrameStage::MainScene),
CameraFrameStagePublishedDepthAliasRole::None);
EXPECT_TRUE(
DoesCameraFrameStageFallbackUseColorOnlySurfaceAccess(
CameraFrameStage::PostProcess));
@@ -102,6 +108,7 @@ TEST(CameraFrameRenderGraphStagePolicy_Test, ReportsGraphManagedOutputColorStage
plan.colorChain.usesGraphManagedMainSceneColor = true;
plan.colorChain.postProcess.usesGraphManagedOutputColor = true;
EXPECT_TRUE(plan.UsesGraphManagedOutputColor(CameraFrameStage::MainScene));
EXPECT_TRUE(UsesCameraFrameStageGraphManagedOutputColor(plan, CameraFrameStage::MainScene));
EXPECT_TRUE(UsesCameraFrameStageGraphManagedOutputColor(plan, CameraFrameStage::PostProcess));
EXPECT_FALSE(UsesCameraFrameStageGraphManagedOutputColor(plan, CameraFrameStage::FinalOutput));