rooty/dxvk
1
0
Fork 0
mirror of https://github.com/doitsujin/dxvk.git synced 2025-02-21 13:54:18 +01:00
Code Issues Projects Releases Wiki Activity

[d3d9] Remove a few unnecessary bitmasks

Browse Source
This commit is contained in:
Robin Kertels 2024-10-13 14:39:04 +02:00 committed by Philip Rebohle
parent 421ead5b30
commit f65897be4c
2 changed files with 42 additions and 46 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

75
src/d3d9/d3d9_device.cpp
View File

@ -181,8 +181,6 @@ namespace dxvk {
m_flags.set(D3D9DeviceFlag::DirtySpecializationEntries);
// Bitfields can't be initialized in header.
m_boundRTs = 0;
m_anyColorWrites = 0;
m_activeRTsWhichAreTextures = 0;
m_alphaSwizzleRTs = 0;
m_lastHazardsRT = 0;
@ -1568,7 +1566,6 @@ namespace dxvk {
m_state.renderTargets[RenderTargetIndex] = rt;
UpdateBoundRTs(RenderTargetIndex);
UpdateActiveRTs(RenderTargetIndex);
uint32_t originalAlphaSwizzleRTs = m_alphaSwizzleRTs;
@ -1843,8 +1840,10 @@ namespace dxvk {
// Clear render targets if we need to.
if (Flags & D3DCLEAR_TARGET) {
for (uint32_t rt : bit::BitMask(m_boundRTs)) {
for (uint32_t rt = 0u; rt < m_state.renderTargets.size(); rt++) {
const auto& rts = m_state.renderTargets[rt];
if (rts == nullptr)
continue;
const auto& rtv = rts->GetRenderTargetView(srgb);
if (likely(rtv != nullptr)) {
@ -2222,22 +2221,22 @@ namespace dxvk {
case D3DRS_COLORWRITEENABLE:
if (likely(!old != !Value))
UpdateAnyColorWrites<0>(!!Value);
UpdateAnyColorWrites<0>();
m_flags.set(D3D9DeviceFlag::DirtyBlendState);
break;
case D3DRS_COLORWRITEENABLE1:
if (likely(!old != !Value))
UpdateAnyColorWrites<1>(!!Value);
UpdateAnyColorWrites<1>();
m_flags.set(D3D9DeviceFlag::DirtyBlendState);
break;
case D3DRS_COLORWRITEENABLE2:
if (likely(!old != !Value))
UpdateAnyColorWrites<2>(!!Value);
UpdateAnyColorWrites<2>();
m_flags.set(D3D9DeviceFlag::DirtyBlendState);
break;
case D3DRS_COLORWRITEENABLE3:
if (likely(!old != !Value))
UpdateAnyColorWrites<3>(!!Value);
UpdateAnyColorWrites<3>();
m_flags.set(D3D9DeviceFlag::DirtyBlendState);
break;
@ -3618,8 +3617,15 @@ namespace dxvk {
// If we have any RTs we would have bound to the the FB
// not in the new shader mask, mark the framebuffer as dirty
// so we unbind them.
uint32_t oldUseMask = m_boundRTs & m_anyColorWrites & m_psShaderMasks.rtMask;
uint32_t newUseMask = m_boundRTs & m_anyColorWrites & newShaderMasks.rtMask;
uint32_t boundMask = 0u;
uint32_t anyColorWriteMask = 0u;
for (uint32_t i = 0u; i < m_state.renderTargets.size(); i++) {
boundMask |= HasRenderTargetBound(i) << i;
anyColorWriteMask |= (m_state.renderStates[ColorWriteIndex(i)] != 0) << i;
}
uint32_t oldUseMask = boundMask & anyColorWriteMask & m_psShaderMasks.rtMask;
uint32_t newUseMask = boundMask & anyColorWriteMask & newShaderMasks.rtMask;
if (oldUseMask != newUseMask)
m_flags.set(D3D9DeviceFlag::DirtyFramebuffer);
@ -5940,42 +5946,25 @@ namespace dxvk {
}
inline void D3D9DeviceEx::UpdateBoundRTs(uint32_t index) {
const uint32_t bit = 1 << index;
m_boundRTs &= ~bit;
if (m_state.renderTargets[index] != nullptr &&
!m_state.renderTargets[index]->IsNull())
m_boundRTs |= bit;
}
inline void D3D9DeviceEx::UpdateActiveRTs(uint32_t index) {
const uint32_t bit = 1 << index;
m_activeRTsWhichAreTextures &= ~bit;
if ((m_boundRTs & bit) != 0 &&
if (HasRenderTargetBound(index) &&
m_state.renderTargets[index]->GetBaseTexture() != nullptr &&
m_anyColorWrites & bit)
m_state.renderStates[ColorWriteIndex(index)] != 0)
m_activeRTsWhichAreTextures |= bit;
UpdateActiveHazardsRT(bit);
}
template <uint32_t Index>
inline void D3D9DeviceEx::UpdateAnyColorWrites(bool has) {
const uint32_t bit = 1 << Index;
m_anyColorWrites &= ~bit;
if (has)
m_anyColorWrites |= bit;
inline void D3D9DeviceEx::UpdateAnyColorWrites() {
// The 0th RT is always bound.
if (Index == 0 || m_boundRTs & bit) {
if (m_boundRTs & bit) {
bool bound = HasRenderTargetBound(Index);
if (Index == 0 || bound) {
if (bound) {
m_flags.set(D3D9DeviceFlag::DirtyFramebuffer);
}
@ -6336,9 +6325,13 @@ namespace dxvk {
// if the RT has the same size or is bigger than the smallest active RT.
uint32_t boundMask = 0u;
uint32_t anyColorWriteMask = 0u;
uint32_t limitsRenderAreaMask = 0u;
VkExtent2D renderArea = { ~0u, ~0u };
for (uint32_t i : bit::BitMask(m_boundRTs)) {
for (uint32_t i = 0u; i < m_state.renderTargets.size(); i++) {
if (m_state.renderTargets[i] == nullptr)
continue;
const DxvkImageCreateInfo& rtImageInfo = m_state.renderTargets[i]->GetCommonTexture()->GetImage()->info();
// Dont bind it if the sample count doesnt match
@ -6359,9 +6352,11 @@ namespace dxvk {
// It will only get bound if its not smaller than the others.
// So RTs with a disabled color write mask will never impact the render area.
if (!(m_anyColorWrites & (1 << i)))
if (m_state.renderStates[ColorWriteIndex(i)] == 0)
continue;
anyColorWriteMask |= 1 << i;
if (rtExtent.width < renderArea.width && rtExtent.height < renderArea.height) {
// It's smaller on both axis, so the previous RTs no longer limit the size
limitsRenderAreaMask = 1 << i;
@ -6396,7 +6391,7 @@ namespace dxvk {
// We only need to skip binding the RT if it would shrink the render area
// despite not having color writes enabled,
// otherwise we might end up with unnecessary render pass spills
boundMask &= (m_anyColorWrites | ~limitsRenderAreaMask);
boundMask &= (anyColorWriteMask | ~limitsRenderAreaMask);
for (uint32_t i : bit::BitMask(boundMask)) {
attachments.color[i] = {
m_state.renderTargets[i]->GetRenderTargetView(srgb),
@ -8149,10 +8144,10 @@ namespace dxvk {
UpdatePixelBoolSpec(0u);
UpdateCommonSamplerSpec(0u, 0u, 0u);
UpdateAnyColorWrites<0>(true);
UpdateAnyColorWrites<1>(true);
UpdateAnyColorWrites<2>(true);
UpdateAnyColorWrites<3>(true);
UpdateAnyColorWrites<0>();
UpdateAnyColorWrites<1>();
UpdateAnyColorWrites<2>();
UpdateAnyColorWrites<3>();
SetIndices(nullptr);
for (uint32_t i = 0; i < caps::MaxStreams; i++) {

13
src/d3d9/d3d9_device.h
View File

@ -796,12 +796,10 @@ namespace dxvk {
void EndFrame();
void UpdateBoundRTs(uint32_t index);
void UpdateActiveRTs(uint32_t index);
template <uint32_t Index>
void UpdateAnyColorWrites(bool has);
void UpdateAnyColorWrites();
void UpdateActiveTextures(uint32_t index, DWORD combinedUsage);
@ -1301,7 +1299,7 @@ namespace dxvk {
bool IsTextureBoundAsAttachment(const D3D9CommonTexture* pTexture) const {
if (unlikely(pTexture->IsRenderTarget())) {
for (uint32_t i : bit::BitMask(m_boundRTs)) {
for (uint32_t i = 0u; i < m_state.renderTargets.size(); i++) {
auto texInfo = m_state.renderTargets[i]->GetCommonTexture();
if (unlikely(texInfo == pTexture)) {
return true;
@ -1316,6 +1314,11 @@ namespace dxvk {
return false;
}
inline bool HasRenderTargetBound(uint32_t Index) const {
return m_state.renderTargets[Index] != nullptr
&& !m_state.renderTargets[Index]->IsNull();
}
Com<D3D9InterfaceEx> m_parent;
D3DDEVTYPE m_deviceType;
HWND m_window;
@ -1392,8 +1395,6 @@ namespace dxvk {
uint32_t m_dirtySamplerStates = 0;
uint32_t m_dirtyTextures = 0;
uint32_t m_boundRTs : 4;
uint32_t m_anyColorWrites : 4;
uint32_t m_activeRTsWhichAreTextures : 4;
uint32_t m_alphaSwizzleRTs : 4;
uint32_t m_lastHazardsRT : 4;