[llvm] [NFC] Refactoring MCDXBC to support out of order storage of root parameters (PR #137284)

[via llvm-commits]

================
@@ -274,27 +274,39 @@ void DXContainerWriter::writeParts(raw_ostream &OS) {
       RS.StaticSamplersOffset = P.RootSignature->StaticSamplersOffset;
 
       for (const auto &Param : P.RootSignature->Parameters) {
-        mcdxbc::RootParameter NewParam;
-        NewParam.Header = dxbc::RootParameterHeader{
-            Param.Type, Param.Visibility, Param.Offset};
+        auto Header = dxbc::RootParameterHeader{Param.Type, Param.Visibility,
+                                                Param.Offset};
 
         switch (Param.Type) {
         case llvm::to_underlying(dxbc::RootParameterType::Constants32Bit):
-          NewParam.Constants.Num32BitValues = Param.Constants.Num32BitValues;
-          NewParam.Constants.RegisterSpace = Param.Constants.RegisterSpace;
-          NewParam.Constants.ShaderRegister = Param.Constants.ShaderRegister;
+          dxbc::RootConstants Constants;
+          Constants.Num32BitValues = Param.Constants.Num32BitValues;
+          Constants.RegisterSpace = Param.Constants.RegisterSpace;
+          Constants.ShaderRegister = Param.Constants.ShaderRegister;
+          RS.ParametersContainer.addParameter(Header, Constants);
----------------
joaosaffran wrote:

I personally prefer methods with less parameters, since I found those easier to use. I could rename `addParameter` to `addConstant` and `addDescriptor`, but I would keep the parameters types:

```C++
addConstant(dxbc::RootParameterHeader, dxbc::RootConstant);
addDescriptor(dxbc::RootParameterHeader, dxbc::RootDescriptor);

```

https://github.com/llvm/llvm-project/pull/137284