Skip to content

Add DirectSR (D3D12) super resolution implementation#780

Open
MikhailGorobets wants to merge 3 commits intoDiligentGraphics:super_resolution2from
MikhailGorobets:super_resolution2
Open

Add DirectSR (D3D12) super resolution implementation#780
MikhailGorobets wants to merge 3 commits intoDiligentGraphics:super_resolution2from
MikhailGorobets:super_resolution2

Conversation

@MikhailGorobets
Copy link
Contributor

No description provided.

}
}

IMPLEMENT_QUERY_INTERFACE_IN_PLACE(IID_SuperResolution, TBase)

Check warning

Code scanning / CodeQL

Local variable address stored in non-local memory Warning

A stack address which arrived via a
parameter
may be assigned to a non-local variable.
{
return const_cast<IReferenceCounters*>(static_cast<const IReferenceCounters*>(&m_RefCounters));
}
IMPLEMENT_QUERY_INTERFACE_IN_PLACE(IID_SuperResolutionFactory, TBase)

Check warning

Code scanning / CodeQL

Local variable address stored in non-local memory Warning

A stack address which arrived via a
parameter
may be assigned to a non-local variable.
Comment on lines +151 to +161
switch (Backend)
{
#if D3D12_SUPPORTED
case SUPER_RESOLUTION_BACKEND_D3D12_DSR:
GetSourceSettingsD3D12(m_pDSRDevice, Attribs, Settings);
break;
#endif
default:
LOG_WARNING_MESSAGE("Unknown super resolution backend");
break;
}

Check notice

Code scanning / CodeQL

No trivial switch statements Note

This switch statement should either handle more cases, or be rewritten as an if statement.
Comment on lines +181 to +191
switch (Backend)
{
#if D3D12_SUPPORTED
case SUPER_RESOLUTION_BACKEND_D3D12_DSR:
CreateSuperResolutionD3D12(m_pDevice, m_pDSRDevice, Desc, ppUpscaler);
break;
#endif
default:
LOG_ERROR_MESSAGE("Unknown super resolution backend");
break;
}

Check notice

Code scanning / CodeQL

No trivial switch statements Note

This switch statement should either handle more cases, or be rewritten as an if statement.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant