[llvm] [HLSL] Add descriptor table metadata parsing (PR #142492)

via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 5 23:56:45 PDT 2025


================
@@ -236,11 +323,84 @@ static bool verifyRegisterValue(uint32_t RegisterValue) {
 // This Range is reserverved, therefore invalid, according to the spec
 // https://github.com/llvm/wg-hlsl/blob/main/proposals/0002-root-signature-in-clang.md#all-the-values-should-be-legal
 static bool verifyRegisterSpace(uint32_t RegisterSpace) {
-  return !(RegisterSpace >= 0xFFFFFFF0 && RegisterSpace <= 0xFFFFFFFF);
+  return !(RegisterSpace >= 0xFFFFFFF0 && RegisterSpace < 0xFFFFFFFF);
----------------
joaosaffran wrote:

You are correct, sorry, copied the example from here: https://github.com/llvm/wg-hlsl/blob/main/proposals/0002-root-signature-in-clang.md#descriptor-ranges, that make me confused. Will fix this code, and the spec

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


More information about the llvm-commits mailing list