[llvm-branch-commits] [clang] [llvm] [DirectX] Updating Root Signature Metadata to contain Static Sampler flags (PR #160210)

[Finn Plummer via llvm-branch-commits]

================
@@ -212,6 +212,7 @@ MDNode *MetadataBuilder::BuildStaticSampler(const 
StaticSampler &Sampler) {
       ConstantAsMetadata::get(Builder.getInt32(Sampler.Space)),
       ConstantAsMetadata::get(
           Builder.getInt32(to_underlying(Sampler.Visibility))),
+      ConstantAsMetadata::get(Builder.getInt32(0)),
----------------
inbelic wrote:

Maybe this is just a temporary measure? If so, I think we can move all of the 
lexing/parsing components to a stacked pr. Otherwise, we have no testing that 
it is working
```suggestion
      ConstantAsMetadata::get(Builder.getInt32(Sample.Flags)),
```

https://github.com/llvm/llvm-project/pull/160210
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits