Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added implementation for ID3D12FunctionReflection1::GetDesc1 #6827

Open
wants to merge 65 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
65 commits
Select commit Hold shift + click to select a range
b9608d6
Added implementation for ID3D12FunctionReflection::GetDesc1
Nielsbishere Jul 25, 2024
41ba55c
Removed unused variable
Nielsbishere Jul 25, 2024
365edc2
Added versioning to function reflection and library reflection
Nielsbishere Jul 26, 2024
32db08b
Fixed missing ID3D12LibraryReflection check
Nielsbishere Jul 26, 2024
5d26589
Fixed QueryInterface in DxilLibraryReflection
Nielsbishere Jul 26, 2024
a42939f
Implemented GetInputNode/GetOutputNode, changed formatting a bit, add…
Nielsbishere Jul 26, 2024
52aa9d9
Added safety check to CFunctionReflection::GetDesc1 and renamed D3D12…
Nielsbishere Jul 26, 2024
7e39b81
Formatted with clang-format
Nielsbishere Jul 26, 2024
0a78e0f
Update to latest DirectX-Headers
Nielsbishere Jul 31, 2024
9814065
DirectX-Headers is now required on Windows too, since otherwise it's …
Nielsbishere Jul 31, 2024
a0bb8b9
Implemented GetWaveSize for DxilContainerReflection.cpp
Nielsbishere Aug 5, 2024
d23fcd2
Merge branch 'feat_query_all_function_properties3' of https://github.…
Nielsbishere Aug 5, 2024
2b92066
WSL Stubs shouldn't be included by default anymore for linux, since d…
Nielsbishere Aug 5, 2024
666dfae
Fixed formatting issue
Nielsbishere Aug 5, 2024
8be0aae
Apparently WSL stubs were not the issue on the remote
Nielsbishere Aug 5, 2024
895f80a
Changed ID3D12ShaderReflection to ID3D12ShaderReflection1 and impleme…
Nielsbishere Aug 5, 2024
5676799
Reformat to align to styleguide
Nielsbishere Aug 5, 2024
6bc299e
Amplification and mesh shaders have thread count too
Nielsbishere Aug 5, 2024
8b97460
Fixed formatting
Nielsbishere Aug 7, 2024
2109491
Added support for viewing descs in dxa.
Nielsbishere Aug 7, 2024
6fbe8a9
Merge branch 'feat_query_all_function_properties3' of https://github.…
Nielsbishere Aug 7, 2024
ea62c4b
Exposed compute shader WaveSizeMin/Max/Preferred and mesh/amplificati…
Nielsbishere Aug 7, 2024
530970c
Formatting
Nielsbishere Aug 7, 2024
3d777a0
Made sure guids are taken from DirectX-Headers d3d12.h rather than el…
Nielsbishere Aug 7, 2024
4a4ce62
Merge trunk into branch
Nielsbishere Aug 7, 2024
2457465
Undid breaking changes. WIP
Nielsbishere Aug 7, 2024
5a60043
Added predeclarations for LibraryReflection1 and ShaderReflection1. U…
Nielsbishere Aug 7, 2024
9eca17a
Windows: wsl stubs aren't needed. Now requiring DirectX-Headers to en…
Nielsbishere Aug 7, 2024
c88a1a6
DirectX-Headers is now always required and removed config file
Nielsbishere Aug 7, 2024
a702d2c
Fixed compile issue due to missing [[fallthrough]]; Fixed duplicate d…
Nielsbishere Aug 7, 2024
0a87922
Update dependency, should resolve build issue on unix
Nielsbishere Aug 7, 2024
506a167
Update dependency, should hopefully fix build errors
Nielsbishere Aug 8, 2024
988ada1
Formatting. Also fixed unit test that relied on the ShaderKind of 'No…
Nielsbishere Aug 8, 2024
7c45598
Hopefully fixed windows build
Nielsbishere Aug 8, 2024
32ac5ec
Seems like ExecutionTest.cpp still has an issue with finding GUIDs th…
Nielsbishere Aug 8, 2024
6defb75
Maybe d3d12.h is the only one that can be used with INITGUID
Nielsbishere Aug 8, 2024
fe507da
Undid removal of definition of ID3D12SDKConfiguration UUIDs, which ar…
Nielsbishere Aug 8, 2024
248ce58
Fix memory leak that somehow was only detected on linux but only on r…
Nielsbishere Aug 8, 2024
ac6c505
dxc/WinAdapter.h now extends DirectX-Headers/WSL's winAdapter.h, to a…
Nielsbishere Aug 9, 2024
903325e
Restored most headers to what they used to be by adding the same defi…
Nielsbishere Aug 9, 2024
ae5c687
Fixed BOOL issues and compile issues on linux
Nielsbishere Aug 9, 2024
dc19327
Fixed windows build
Nielsbishere Aug 9, 2024
631d446
Clean up for PR
Nielsbishere Aug 9, 2024
6e41faf
Undid some changes for PR
Nielsbishere Aug 9, 2024
18a4894
DirectX-Headers is now required for windows
Nielsbishere Aug 9, 2024
d1d195a
Formatting
Nielsbishere Aug 9, 2024
d82e56d
Some compilers don't seem to like missing endline at the end of a fil…
Nielsbishere Aug 9, 2024
9d72073
Merge and fix some linux/mac issues
Nielsbishere Aug 9, 2024
7883cda
Clang still needs __wsl_stub_uuidof, even though it has builtin suppo…
Nielsbishere Aug 9, 2024
15d406c
Formatting
Nielsbishere Aug 9, 2024
21a5225
Clang didn't have guid_from_string yet
Nielsbishere Aug 9, 2024
e0272d7
Removed duplicate defintions, since now d3d12.h from DirectX-Headers …
Nielsbishere Aug 9, 2024
06a33f8
Merge branch update_directx_headers into feat_query_all_function_prop…
Nielsbishere Aug 21, 2024
0d91e56
DirectX-Headers wasn't properly up to date
Nielsbishere Aug 21, 2024
e000305
Fixed issue with failed merge in DirectX-Headers
Nielsbishere Aug 21, 2024
0432255
Fixed build if interface is defined by winadapter
Nielsbishere Aug 21, 2024
cca5f93
Merge branch 'main' of https://github.com/Microsoft/DirectXShaderComp…
Nielsbishere Aug 21, 2024
042ed57
Merge upstream back into branch
Nielsbishere Aug 21, 2024
4faa40c
Added unit tests by modifying existing unit tests to include reflecti…
Nielsbishere Aug 21, 2024
74e793d
Apparently even though DXA runs fine, the file checker uses something…
Nielsbishere Aug 21, 2024
98774b8
Fixed a few tests; replaced LibraryReflection with LibraryReflection1…
Nielsbishere Aug 21, 2024
4d33ef8
Fixed unit tests hopefully
Nielsbishere Aug 22, 2024
828e48e
Redid formatting and fixed a mistake
Nielsbishere Aug 22, 2024
4c81c8c
Fixed the last remaining unit tests
Nielsbishere Aug 22, 2024
7ed6944
Apparently the featureflag with SM6.8 isn't set for SM6.7 when sm65_m…
Nielsbishere Aug 22, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 9 additions & 5 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -671,6 +671,15 @@ if(LLVM_INCLUDE_TESTS AND WIN32)
add_definitions(/DMSFT_SUPPORTS_CHILD_PROCESSES)
endif()

if(EXISTS "${LLVM_MAIN_SRC_DIR}/external")
add_subdirectory(external) # SPIRV change
endif()
if(WIN32)
include_directories(${DIRECTX_HEADER_INCLUDE_DIR}/directx)
else()
include_directories(AFTER ${DIRECTX_HEADER_INCLUDE_DIR}/wsl/stubs ${DIRECTX_HEADER_INCLUDE_DIR}/directx)
endif()

# Put this before tblgen. Else we have a circular dependence.
add_subdirectory(lib/Support)
add_subdirectory(lib/MSSupport) # HLSL Change
Expand All @@ -686,11 +695,6 @@ add_subdirectory(include/dxc)
# really depend on anything else in the build it is safe.
list(APPEND LLVM_COMMON_DEPENDS HCTGen)

if(EXISTS "${LLVM_MAIN_SRC_DIR}/external")
add_subdirectory(external) # SPIRV change
endif()
include_directories(AFTER ${DIRECTX_HEADER_INCLUDE_DIR}/directx ${DIRECTX_HEADER_INCLUDE_DIR}/wsl/stubs)

# HLSL - Change End

add_subdirectory(lib)
Expand Down
12 changes: 5 additions & 7 deletions external/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,11 @@ endif (NOT HLSL_ENABLE_DEBUG_ITERATORS)

# Need DirectX-Headers module if not on windows
if (NOT DIRECTX_HEADER_INCLUDE_DIR)
if (NOT WIN32)
if (IS_DIRECTORY "${DXC_EXTERNAL_ROOT_DIR}/DirectX-Headers")
set(DIRECTX_HEADER_INCLUDE_DIR ${DXC_EXTERNAL_ROOT_DIR}/DirectX-Headers/include PARENT_SCOPE)
else()
message(FATAL_ERROR "DirectX-Headers was not found - required for reflection support on *nix see https://github.com/microsoft/DirectX-Headers")
endif()
endif (NOT WIN32)
if (IS_DIRECTORY "${DXC_EXTERNAL_ROOT_DIR}/DirectX-Headers")
set(DIRECTX_HEADER_INCLUDE_DIR ${DXC_EXTERNAL_ROOT_DIR}/DirectX-Headers/include PARENT_SCOPE)
else()
message(FATAL_ERROR "DirectX-Headers was not found - required for reflection support on *nix see https://github.com/microsoft/DirectX-Headers")
endif()
endif(NOT DIRECTX_HEADER_INCLUDE_DIR)

# Enabling SPIR-V codegen requires SPIRV-Headers for spirv.hpp and
Expand Down
2 changes: 1 addition & 1 deletion external/DirectX-Headers
Submodule DirectX-Headers updated 56 files
+23 −0 .gitignore
+43 −17 CMakeLists.txt
+187 −0 CMakePresets.json
+10 −0 CODE_OF_CONDUCT.md
+41 −41 SECURITY.md
+10 −0 cmake/DirectX-Headers.pc.in
+23 −0 cmake/JoinPaths.cmake
+32 −2 googletest/CMakeLists.txt
+184 −137 googletest/MockDevice.hpp
+2 −0 googletest/feature_support_test.cpp
+13 −0 googletest/meson.build
+2,634 −0 include/directx/D3D12TokenizedProgramFormat.hpp
+2,651 −0 include/directx/DirectML.h
+16,396 −6,705 include/directx/d3d12.h
+946 −103 include/directx/d3d12.idl
+167 −7 include/directx/d3d12compatibility.h
+11 −3 include/directx/d3d12compatibility.idl
+467 −18 include/directx/d3d12sdklayers.h
+136 −4 include/directx/d3d12sdklayers.idl
+248 −5 include/directx/d3d12shader.h
+934 −68 include/directx/d3d12video.h
+750 −68 include/directx/d3d12video.idl
+64 −4 include/directx/d3dcommon.h
+78 −4 include/directx/d3dcommon.idl
+13 −5,039 include/directx/d3dx12.h
+192 −0 include/directx/d3dx12_barriers.h
+1,138 −0 include/directx/d3dx12_check_feature_support.h
+1,537 −0 include/directx/d3dx12_core.h
+12 −0 include/directx/d3dx12_default.h
+1,497 −0 include/directx/d3dx12_pipeline_state_stream.h
+124 −0 include/directx/d3dx12_property_format_table.h
+105 −0 include/directx/d3dx12_render_pass.h
+602 −0 include/directx/d3dx12_resource_helpers.h
+1,227 −0 include/directx/d3dx12_root_signature.h
+2,210 −0 include/directx/d3dx12_state_object.h
+184 −3 include/directx/dxcore_interface.h
+1 −0 include/directx/dxgicommon.idl
+2 −0 include/directx/dxgiformat.h
+2 −0 include/directx/dxgiformat.idl
+77 −6 include/dxguids/dxguids.h
+400 −0 include/wsl/stubs/basetsd.h
+0 −0 include/wsl/stubs/oaidl.h
+0 −0 include/wsl/stubs/ocidl.h
+1 −1 include/wsl/stubs/rpc.h
+73 −1 include/wsl/stubs/rpcndr.h
+3 −0 include/wsl/stubs/unknwn.h
+124 −0 include/wsl/stubs/unknwnbase.h
+1 −400 include/wsl/winadapter.h
+7 −7 include/wsl/wrladapter.h
+33 −4 meson.build
+2,445 −0 src/d3dx12_property_format_table.cpp
+2 −3 src/dxguids.cpp
+48 −4 test/CMakeLists.txt
+98 −6 test/feature_check_test.cpp
+11 −5 test/meson.build
+16 −3 test/test.cpp
2 changes: 1 addition & 1 deletion include/dxc/Support/D3DReflection.h
Original file line number Diff line number Diff line change
Expand Up @@ -22,5 +22,5 @@
#undef interface
#pragma GCC diagnostic pop
#else
#include <d3d12shader.h>
#include "d3d12shader.h"
#endif
12 changes: 7 additions & 5 deletions include/dxc/Support/Unicode.h
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
#include <string>

#ifdef _WIN32
typedef int *LPBOOL;
#include <specstrings.h>
#else
// MultiByteToWideChar which is a Windows-specific method.
Expand All @@ -30,7 +31,7 @@ int WideCharToMultiByte(uint32_t CodePage, uint32_t dwFlags,
const wchar_t *lpWideCharStr, int cchWideChar,
char *lpMultiByteStr, int cbMultiByte,
const char *lpDefaultChar = nullptr,
bool *lpUsedDefaultChar = nullptr);
LPBOOL lpUsedDefaultChar = nullptr);
#endif // _WIN32

namespace Unicode {
Expand All @@ -54,14 +55,15 @@ typedef char acp_char;
typedef char ccp_char;

bool UTF8ToConsoleString(const char *text, size_t textLen, std::string *pValue,
bool *lossy);
LPBOOL lossy);

bool UTF8ToConsoleString(const char *text, std::string *pValue, bool *lossy);
bool UTF8ToConsoleString(const char *text, std::string *pValue, LPBOOL lossy);

bool WideToConsoleString(const wchar_t *text, size_t textLen,
std::string *pValue, bool *lossy);
std::string *pValue, LPBOOL lossy);

bool WideToConsoleString(const wchar_t *text, std::string *pValue, bool *lossy);
bool WideToConsoleString(const wchar_t *text, std::string *pValue,
LPBOOL lossy);

bool UTF8ToWideString(const char *pUTF8, std::wstring *pWide);

Expand Down
16 changes: 15 additions & 1 deletion include/dxc/Test/D3DReflectionDumper.h
Original file line number Diff line number Diff line change
Expand Up @@ -38,18 +38,32 @@ class D3DReflectionDumper : public DumpContext {
void Dump(D3D12_SHADER_BUFFER_DESC &Desc);
void Dump(D3D12_SHADER_INPUT_BIND_DESC &resDesc);
void Dump(D3D12_SIGNATURE_PARAMETER_DESC &elDesc);
void Dump(D3D12_NODE_SHADER_DESC &Desc);
void Dump(D3D12_HULL_SHADER_DESC &Desc);
void Dump(D3D12_COMPUTE_SHADER_DESC &Desc);
void Dump(D3D12_MESH_SHADER_DESC &Desc);
void Dump(D3D12_GEOMETRY_SHADER_DESC &Desc);
void Dump(D3D12_DOMAIN_SHADER_DESC &Desc);
void Dump(D3D12_NODE_ID_DESC &Desc, const char *name);
void Dump(D3D12_NODE_DESC &Desc);
void Dump(D3D12_SHADER_DESC &Desc);
void Dump(D3D12_FUNCTION_DESC &Desc);
void Dump(D3D12_LIBRARY_DESC &Desc);

void Dump(D3D12_FUNCTION_DESC1 &Desc);

void Dump(ID3D12ShaderReflectionType *pType);
void Dump(ID3D12ShaderReflectionVariable *pVar);

void Dump(ID3D12ShaderReflectionConstantBuffer *pCBReflection);

void Dump(ID3D12ShaderReflection *pShaderReflection);
void Dump(ID3D12ShaderReflection *pShaderReflection,
BOOL printHeader = false);
void Dump(ID3D12ShaderReflection1 *pShaderReflection);
void Dump(ID3D12FunctionReflection *pFunctionReflection);
void Dump(ID3D12FunctionReflection1 *pFunctionReflection);
void Dump(ID3D12LibraryReflection *pLibraryReflection);
void Dump(ID3D12LibraryReflection1 *pLibraryReflection);
};

} // namespace dump
Expand Down
2 changes: 2 additions & 0 deletions include/dxc/Test/D3DReflectionStrings.h
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ LPCSTR ToString(D3D_PARAMETER_FLAGS Flag);
LPCSTR ToString(D3D_NAME Name);
LPCSTR ToString(D3D_REGISTER_COMPONENT_TYPE CompTy);
LPCSTR ToString(D3D_MIN_PRECISION MinPrec);
LPCSTR ToString(D3D12_NODE_LAUNCH_TYPE NodeLaunchType);
LPCSTR ToString(D3D12_DISPATCH_COMPONENT_TYPE DispatchComponentType);
LPCSTR CompMaskToString(unsigned CompMask);

// These macros declare the ToString functions for DXC types
Expand Down
Loading
Loading