[llvm-branch-commits] [llvm] [DirectX] Updating Root Signature YAML representation to use Enums instead of uint (PR #154827)

Justin Bogner via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Thu Aug 28 13:25:38 PDT 2025


================
@@ -276,10 +276,6 @@ void DXContainerWriter::writeParts(raw_ostream &OS) {
       for (DXContainerYAML::RootParameterLocationYaml &L :
            P.RootSignature->Parameters.Locations) {
 
-        assert(dxbc::isValidParameterType(L.Header.Type) &&
-               "invalid DXContainer YAML");
-        assert(dxbc::isValidShaderVisibility(L.Header.Visibility) &&
-               "invalid DXContainer YAML");
         dxbc::RootParameterType Type = dxbc::RootParameterType(L.Header.Type);
         dxbc::ShaderVisibility Visibility =
             dxbc::ShaderVisibility(L.Header.Visibility);
----------------
bogner wrote:

The casts are no longer needed, since these are already the right type now. Here and throughout this file.

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


More information about the llvm-branch-commits mailing list