[clang] [llvm] [X86][AVX10.2] Support YMM rounding new instructions (PR #101825)
Shengchen Kan via cfe-commits
cfe-commits at lists.llvm.org
Sat Aug 3 07:13:37 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!");
----------------
KanRobert wrote:
I suggest not using the tricky `-1`, it's not robust b/c the suffix `U` can represent unsigned long long (https://en.cppreference.com/w/cpp/language/integer_literal)
https://github.com/llvm/llvm-project/pull/101825
More information about the cfe-commits
mailing list