Skip to content

[NFC] Edit DXIL module flag comments for clarity #7271

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.

Sign up for GitHub

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
+15 −5
Open

[NFC] Edit DXIL module flag comments for clarity #7271

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1f20688
[NFC] Edit shader flag comments for clarity
Icohedron Mar 28, 2025
151e318
Add a comment to describe the bitmask and to refer to GetFeatureInfo(…
Icohedron Mar 29, 2025
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
2 changes: 1 addition & 1 deletion include/dxc/DXIL/DxilConstants.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ inline bool UpdateToMaxOfVersions(unsigned &major, unsigned &minor,
return false;
}

// Shader flags.
// Global shader flags
const unsigned kDisableOptimizations =
0x00000001; // D3D11_1_SB_GLOBAL_FLAG_SKIP_OPTIMIZATION
const unsigned kDisableMathRefactoring =
Expand Down
11 changes: 7 additions & 4 deletions include/dxc/DXIL/DxilShaderFlags.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -227,17 +227,22 @@ class ShaderFlags {
m_bDisableMathRefactoring : 1; //~D3D10_SB_GLOBAL_FLAG_REFACTORING_ALLOWED
unsigned
m_bEnableDoublePrecision : 1; // D3D11_SB_GLOBAL_FLAG_ENABLE_DOUBLE_PRECISION_FLOAT_OPS
// SHADER_FEATURE_DOUBLES
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DxilShaderFlags.cpp:L57

unsigned
m_bForceEarlyDepthStencil : 1; // D3D11_SB_GLOBAL_FLAG_FORCE_EARLY_DEPTH_STENCIL

// Bit: 4
unsigned
m_bEnableRawAndStructuredBuffers : 1; // D3D11_SB_GLOBAL_FLAG_ENABLE_RAW_AND_STRUCTURED_BUFFERS
unsigned
m_bLowPrecisionPresent : 1; // D3D11_1_SB_GLOBAL_FLAG_ENABLE_MINIMUM_PRECISION
m_bLowPrecisionPresent : 1; // AND ~m_bUseNativeLowPrecision => D3D11_1_SB_GLOBAL_FLAG_ENABLE_MINIMUM_PRECISION
Copy link
Collaborator Author

@Icohedron Icohedron Mar 28, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DxilShaderFlags.cpp:L215

// AND ~m_bUseNativeLowPrecision => SHADER_FEATURE_MINIMUM_PRECISION
// AND m_bUseNativeLowPrecision => SHADER_FEATURE_NATIVE_LOW_PRECISION
Comment on lines +246 to +247
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DxilShaderFlags.cpp:L58
DxilShaderFlags.cpp:L61

unsigned
m_bEnableDoubleExtensions : 1; // D3D11_1_SB_GLOBAL_FLAG_ENABLE_DOUBLE_EXTENSIONS
// SHADER_FEATURE_11_1_DOUBLE_EXTENSIONS
Copy link
Collaborator Author

@Icohedron Icohedron Mar 28, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DxilShaderFlags.cpp:L65

unsigned m_bEnableMSAD : 1; // D3D11_1_SB_GLOBAL_FLAG_ENABLE_SHADER_EXTENSIONS
// SHADER_FEATURE_11_1_SHADER_EXTENSIONS
Copy link
Collaborator Author

@Icohedron Icohedron Mar 28, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DxilShaderFlags.cpp:L79


// Bit: 8
unsigned m_bAllResourcesBound : 1; // D3D12_SB_GLOBAL_FLAG_ALL_RESOURCES_BOUND
Expand All @@ -253,8 +258,6 @@ class ShaderFlags {
m_bUAVLoadAdditionalFormats : 1; // SHADER_FEATURE_TYPED_UAV_LOAD_ADDITIONAL_FORMATS
unsigned
m_bLevel9ComparisonFiltering : 1; // SHADER_FEATURE_LEVEL_9_COMPARISON_FILTERING
// SHADER_FEATURE_11_1_SHADER_EXTENSIONS
// shared with EnableMSAD
Copy link
Collaborator Author

@Icohedron Icohedron Mar 28, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed these two lines because I see m_bLevel9ComparisonFiltering only being used to set SHADER_FEATURE_LEVEL_9_COMPARISON_FILTERING.
DxilShaderFlags.cpp:L88

unsigned m_b64UAVs : 1; // SHADER_FEATURE_64_UAVS

// Bit: 16
Expand All @@ -277,7 +280,7 @@ class ShaderFlags {
unsigned m_bBarycentrics : 1; // SHADER_FEATURE_BARYCENTRICS

// SM 6.2+
unsigned m_bUseNativeLowPrecision : 1;
unsigned m_bUseNativeLowPrecision : 1; // see m_bLowPrecisionPresent for uses

// SM 6.4+
// Bit: 24
Expand Down
Loading