[clang] [llvm] [X86][AVX10.2] Support YMM rounding new instructions (PR #101825)

Phoebe Wang via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 3 07:28:27 PDT 2024


================
@@ -785,9 +785,9 @@ void DisassemblerTables::emitModRMDecision(raw_ostream &o1, raw_ostream &o2,
     break;
   }
 
-  // We assume that the index can fit into uint16_t.
-  assert(sEntryNumber < 65536U &&
-         "Index into ModRMDecision is too large for uint16_t!");
+  // We assume that the index can fit into uint32_t.
+  assert(sEntryNumber < -1U &&
+         "Index into ModRMDecision is too large for uint32_t!");
----------------
phoebewang wrote:

`-1U` is wildly used in LLVM code base. I think the document means it can be automatically recognized as `LLU` with the range of constant value, instead of not to use it for small value. See https://godbolt.org/z/9W9srbYYM

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


More information about the llvm-commits mailing list