Skip to content

Commit c3fa785

Browse files
authored
Remove assert.h include from DxilPipelineStateValidation.h (#3129)
Also: - fixed size_t to uint32_t error when building with other tools. - removed DxilPipelineStateValidation.h where unused.
1 parent 28499c6 commit c3fa785

9 files changed

Lines changed: 14 additions & 7 deletions

File tree

include/dxc/DxilContainer/DxilPipelineStateValidation.h

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,11 @@
1414

1515
#include <stdint.h>
1616
#include <string.h>
17-
#include "assert.h"
17+
18+
// Don't include assert.h here.
19+
// Since this header is included from multiple environments,
20+
// it is necessary to define assert before this header is included.
21+
// #include <assert.h>
1822

1923
#ifndef UINT_MAX
2024
#define UINT_MAX 0xffffffff
@@ -602,13 +606,13 @@ DxilPipelineStateValidation::CheckedReaderWriter::CheckBounds(size_t size) {
602606
inline bool
603607
DxilPipelineStateValidation::CheckedReaderWriter::IncrementPos(size_t size) {
604608
PSV_RETB(size <= UINT_MAX);
605-
uint32_t uSize = size;
609+
uint32_t uSize = (uint32_t)size;
606610
if (Mode == RWMode::CalcSize) {
607611
PSV_RETB(uSize <= Size + uSize);
608612
Size += uSize;
609613
}
610614
PSV_RETB(CheckBounds(size));
611-
Offset += size;
615+
Offset += uSize;
612616
return true;
613617
}
614618

lib/DxilContainer/DxilContainerAssembler.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232
#include "dxc/Support/WinIncludes.h"
3333
#include "dxc/Support/FileIOHelper.h"
3434
#include "dxc/Support/dxcapi.impl.h"
35+
#include <assert.h> // Needed for DxilPipelineStateValidation.h
3536
#include "dxc/DxilContainer/DxilPipelineStateValidation.h"
3637
#include "dxc/DxilContainer/DxilRuntimeReflection.h"
3738
#include "dxc/DXIL/DxilCounters.h"

lib/DxilRootSignature/DxilRootSignature.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
#include "dxc/DXIL/DxilConstants.h"
1313
#include "dxc/DxilRootSignature/DxilRootSignature.h"
14-
#include "dxc/DxilContainer/DxilPipelineStateValidation.h"
1514
#include "dxc/Support/Global.h"
1615
#include "dxc/Support/WinIncludes.h"
1716
#include "dxc/Support/WinFunctions.h"

lib/DxilRootSignature/DxilRootSignatureConvert.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
#include "dxc/DXIL/DxilConstants.h"
1313
#include "dxc/DxilRootSignature/DxilRootSignature.h"
14-
#include "dxc/DxilContainer/DxilPipelineStateValidation.h"
1514
#include "dxc/Support/Global.h"
1615
#include "dxc/Support/WinIncludes.h"
1716
#include "dxc/Support/WinFunctions.h"

lib/DxilRootSignature/DxilRootSignatureSerializer.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
#include "dxc/DXIL/DxilConstants.h"
1313
#include "dxc/DxilRootSignature/DxilRootSignature.h"
14-
#include "dxc/DxilContainer/DxilPipelineStateValidation.h"
1514
#include "dxc/Support/Global.h"
1615
#include "dxc/Support/WinIncludes.h"
1716
#include "dxc/Support/WinFunctions.h"

lib/DxilRootSignature/DxilRootSignatureValidator.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
#include "dxc/DXIL/DxilConstants.h"
1313
#include "dxc/DxilRootSignature/DxilRootSignature.h"
14-
#include "dxc/DxilContainer/DxilPipelineStateValidation.h"
1514
#include "dxc/Support/Global.h"
1615
#include "dxc/Support/WinIncludes.h"
1716
#include "dxc/Support/WinFunctions.h"
@@ -28,6 +27,9 @@
2827
#include <set>
2928
#include <ios>
3029

30+
#include <assert.h> // Needed for DxilPipelineStateValidation.h
31+
#include "dxc/DxilContainer/DxilPipelineStateValidation.h"
32+
3133
#include "DxilRootSignatureHelper.h"
3234

3335
using namespace llvm;

lib/HLSL/DxilSignatureValidation.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
using namespace hlsl;
1818
using namespace llvm;
1919

20+
#include <assert.h> // Needed for DxilPipelineStateValidation.h
2021
#include "dxc/DxilContainer/DxilPipelineStateValidation.h"
2122
#include <functional>
2223
#include "dxc/HLSL/ViewIDPipelineValidation.inl"

tools/clang/tools/dxcompiler/dxcdisassembler.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
#include "llvm/IR/DebugInfoMetadata.h"
2929
#include "llvm/Support/FormattedStream.h"
3030
#include "llvm/Support/Format.h"
31+
#include <assert.h> // Needed for DxilPipelineStateValidation.h
3132
#include "dxc/DxilContainer/DxilPipelineStateValidation.h"
3233
#include "dxc/DxilContainer/DxilContainer.h"
3334
#include "dxc/DxilContainer/DxilRuntimeReflection.h"

tools/clang/unittests/HLSL/DxilContainerTest.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@
5252
#include "dxc/Support/HLSLOptions.h"
5353
#include "dxc/DxilContainer/DxilContainer.h"
5454
#include "dxc/DxilContainer/DxilRuntimeReflection.h"
55+
#include <assert.h> // Needed for DxilPipelineStateValidation.h
5556
#include "dxc/DxilContainer/DxilPipelineStateValidation.h"
5657
#include "dxc/DXIL/DxilShaderFlags.h"
5758
#include "dxc/DXIL/DxilUtil.h"

0 commit comments

Comments
 (0)