Skip to content

Commit bb623cc

Browse files
committed
[d3d9] Consolidate FFVS dirty flags
No reason to have two when the only thing one of them does is set the other.
1 parent 8486e99 commit bb623cc

1 file changed

Lines changed: 12 additions & 20 deletions

File tree

src/d3d9/d3d9_device.cpp

Lines changed: 12 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,6 @@ namespace dxvk {
138138
D3D9DeviceDirtyFlag::Fog,
139139
D3D9DeviceDirtyFlag::FFVertexData,
140140
D3D9DeviceDirtyFlag::FFVertexBlend,
141-
D3D9DeviceDirtyFlag::FFVertexShader,
142141
D3D9DeviceDirtyFlag::FFPixelShader,
143142
D3D9DeviceDirtyFlag::FFViewport,
144143
D3D9DeviceDirtyFlag::SharedPixelShaderData,
@@ -2220,8 +2219,7 @@ namespace dxvk {
22202219
light.isValid = true;
22212220
light.isEnabled = bool(Enable);
22222221

2223-
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData,
2224-
D3D9DeviceDirtyFlag::FFVertexShader);
2222+
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData);
22252223
return D3D_OK;
22262224
}
22272225

@@ -2433,7 +2431,7 @@ namespace dxvk {
24332431

24342432
case D3DRS_CLIPPLANEENABLE:
24352433
if (!Value != !oldValue)
2436-
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexShader);
2434+
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData);
24372435

24382436
m_dirty.set(D3D9DeviceDirtyFlag::ClipPlanes);
24392437
break;
@@ -2456,7 +2454,7 @@ namespace dxvk {
24562454
case D3DRS_LIGHTING:
24572455
case D3DRS_NORMALIZENORMALS:
24582456
case D3DRS_LOCALVIEWER:
2459-
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexShader);
2457+
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData);
24602458
break;
24612459

24622460
case D3DRS_AMBIENT:
@@ -2479,7 +2477,7 @@ namespace dxvk {
24792477
break;
24802478

24812479
case D3DRS_RANGEFOGENABLE:
2482-
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexShader);
2480+
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData);
24832481
break;
24842482

24852483
case D3DRS_POINTSIZE: {
@@ -2563,14 +2561,14 @@ namespace dxvk {
25632561
break;
25642562

25652563
case D3DRS_VERTEXBLEND:
2566-
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexShader);
2564+
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData);
25672565
break;
25682566

25692567
case D3DRS_INDEXEDVERTEXBLENDENABLE:
25702568
if (CanSWVP() && Value)
25712569
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexBlend);
25722570

2573-
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexShader);
2571+
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData);
25742572
break;
25752573

25762574
case D3DRS_ADAPTIVETESS_Y: {
@@ -3427,7 +3425,7 @@ namespace dxvk {
34273425
|| decl->GetTexcoordMask() != m_state.vertexDecl->GetTexcoordMask();
34283426

34293427
if (dirtyFFShader)
3430-
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexShader);
3428+
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData);
34313429

34323430
const bool wasUsingProgrammableVS = UseProgrammableVS();
34333431

@@ -3439,7 +3437,7 @@ namespace dxvk {
34393437
if (usesProgrammableVS) {
34403438
BindShader<D3D9ShaderType::VertexShader>(GetCommonShader(m_state.vertexShader));
34413439
} else {
3442-
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexShader);
3440+
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData);
34433441
BindFFUbershader<D3D9ShaderType::VertexShader>();
34443442
}
34453443
}
@@ -3560,7 +3558,7 @@ namespace dxvk {
35603558
if (usesProgrammableVS) {
35613559
BindShader<D3D9ShaderType::VertexShader>(GetCommonShader(shader));
35623560
} else if (wasUsingProgrammableVS) {
3563-
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexShader);
3561+
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData);
35643562
BindFFUbershader<D3D9ShaderType::VertexShader>();
35653563
}
35663564

@@ -4681,15 +4679,15 @@ namespace dxvk {
46814679
break;
46824680

46834681
case DXVK_TSS_TEXCOORDINDEX:
4684-
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexShader);
4682+
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData);
46854683
break;
46864684

46874685
case DXVK_TSS_TEXTURETRANSFORMFLAGS:
46884686
m_textureSlotTracking.projected &= ~(1 << Stage);
46894687
if (Value & D3DTTFF_PROJECTED)
46904688
m_textureSlotTracking.projected |= 1 << Stage;
46914689

4692-
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexShader);
4690+
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData);
46934691
m_dirty.set(D3D9DeviceDirtyFlag::FFPixelShader);
46944692
break;
46954693

@@ -8108,12 +8106,6 @@ namespace dxvk {
81088106
vertexBlendMode = D3D9FF_VertexBlendMode_Disabled;
81098107
}
81108108

8111-
// Shader...
8112-
if (m_dirty.test(D3D9DeviceDirtyFlag::FFVertexShader)) {
8113-
m_dirty.clr(D3D9DeviceDirtyFlag::FFVertexShader);
8114-
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData);
8115-
}
8116-
81178109
// Viewport...
81188110
if (hasPositionT && (m_dirty.test(D3D9DeviceDirtyFlag::FFViewport) || m_ffZTest != IsZTestEnabled())) {
81198111
m_dirty.clr(D3D9DeviceDirtyFlag::FFViewport);
@@ -8570,7 +8562,7 @@ namespace dxvk {
85708562
rs[D3DRS_LOCALVIEWER] = TRUE;
85718563
rs[D3DRS_RANGEFOGENABLE] = FALSE;
85728564
rs[D3DRS_NORMALIZENORMALS] = FALSE;
8573-
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexShader);
8565+
m_dirty.set(D3D9DeviceDirtyFlag::FFVertexData);
85748566

85758567
// PS
85768568
rs[D3DRS_SPECULARENABLE] = FALSE;

0 commit comments

Comments
 (0)