================
@@ -704,7 +713,10 @@ void BinaryContext::populateJumpTables() {
uint64_t NextJTAddress = 0;
auto NextJTI = std::next(JTI);
- if (NextJTI != JTE)
+ if (isAArch64()) {
----------------
aaupov wrote:
Thanks for a suggestion, this part is a bit nuanced and needs a comment.
First, there's an implicit assumption that on ARM jump tables are only created
in JumpTableInfoReader, not through instruction sequence/memory analysis as on
X86.
Second, since jump table size is known in advance, NextJTAddress is just used
as end address. It's not actually used to find the next jump table (neither on
X86).
https://github.com/llvm/llvm-project/pull/132114
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
