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

[via llvm-branch-commits]

================
@@ -212,7 +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))};
----------------
joaosaffran wrote:

I think that merge with this temporary fix is better. Since, if we update the 
frontend first, it is going to break the backend, since it still expect 
StaticSampler MD Nodes to contain 14 values and not 15. @bogner what do you 
think is the best direction here?

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