remove version

Author: joaosaffran

llvm/include/llvm/MC/DXContainerRootSignature.h

@@ -15,7 +15,6 @@ class raw_ostream;
 
 namespace mcdxbc {
 struct RootSignatureHeader {
-  uint32_t Version;
   uint32_t Flags;
 
   void swapBytes();

llvm/include/llvm/Object/DXContainer.h

@@ -121,7 +121,6 @@ class RootSignature {
 private:
   StringRef Data;
   uint32_t Size;
-  uint32_t Version;
   uint32_t Flags;
 
 public:
@@ -131,8 +130,6 @@ class RootSignature {
 
   uint32_t getSize() const { return Size; }
 
-  uint32_t getVersion() const { return Version; }
-
   uint32_t getFlags() const { return Flags; }
 };
 

llvm/include/llvm/ObjectYAML/DXContainerYAML.h

@@ -80,7 +80,6 @@ struct RootSignatureDesc {
 
   uint32_t getEncodedFlags();
   uint32_t Size;
-  uint32_t Version;
   uint32_t NumParameters;
   SmallVector<dxbc::RootParameter> Parameters;
 

llvm/lib/MC/DXContainerRootSignature.cpp

@@ -18,6 +18,5 @@ void RootSignatureHeader::write(raw_ostream &OS) {
 
   uint32_t SizeInfo = sizeof(this);
   support::endian::write(OS, SizeInfo, llvm::endianness::little);
-  support::endian::write(OS, Version, llvm::endianness::little);
   support::endian::write(OS, Flags, llvm::endianness::little);
 }

llvm/lib/Object/DXContainer.cpp

@@ -248,9 +248,6 @@ Error DirectX::RootSignature::parse() {
   Size = support::endian::read<uint32_t, llvm::endianness::little>(Current);
   Current += sizeof(uint32_t);
 
-  Version = support::endian::read<uint32_t, llvm::endianness::little>(Current);
-  Current += sizeof(uint32_t);
-
   Flags = support::endian::read<uint32_t, llvm::endianness::little>(Current);
   Current += sizeof(uint32_t);
 

llvm/lib/ObjectYAML/DXContainerEmitter.cpp

@@ -267,7 +267,6 @@ void DXContainerWriter::writeParts(raw_ostream &OS) {
         continue;
 
       mcdxbc::RootSignatureHeader Header;
-      Header.Version = P.RootSignature->Version;
       Header.Flags = P.RootSignature->getEncodedFlags();
 
       Header.write(OS);

llvm/lib/ObjectYAML/DXContainerYAML.cpp

@@ -32,7 +32,7 @@ DXContainerYAML::ShaderFeatureFlags::ShaderFeatureFlags(uint64_t FlagData) {
 
 DXContainerYAML::RootSignatureDesc::RootSignatureDesc(
     const object::DirectX::RootSignature &Data)
-    : Size(Data.getSize()), Version(Data.getVersion()) {
+    : Size(Data.getSize()) {
   uint32_t Flags = Data.getFlags();
 #define ROOT_ELEMENT_FLAG(Num, Val, Str)                                       \
   Val = (Flags & (uint32_t)dxbc::RootElementFlag::Val) > 0;
@@ -210,7 +210,6 @@ void MappingTraits<DXContainerYAML::Signature>::mapping(
 void MappingTraits<DXContainerYAML::RootSignatureDesc>::mapping(
     IO &IO, DXContainerYAML::RootSignatureDesc &S) {
   IO.mapRequired("Size", S.Size);
-  IO.mapRequired("Version", S.Version);
 #define ROOT_ELEMENT_FLAG(Num, Val, Str) IO.mapOptional(#Val, S.Val, false);
 #include "llvm/BinaryFormat/DXContainerConstants.def"
 }

llvm/test/ObjectYAML/DXContainer/RootSignature-Flags.yaml

@@ -13,12 +13,10 @@ Parts:
     Size:            8
     RootSignature:
       Size: 8
-      Version:         1
       AllowInputAssemblerInputLayout: true
 
 #CHECK:    - Name: RTS0
 #CHECK-NEXT: Size: 8
 #CHECK-NEXT: RootSignature:
 #CHECK-NEXT:   Size: 8
-#CHECK-NEXT:   Version: 1
 #CHECK-NEXT:   AllowInputAssemblerInputLayout: true