[llvm] [llvm-debuginfo-analyzer] Fix incorrect REQUIRES in WebAssembly test case (PR #145848)
Carlos Alberto Enciso via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 25 23:13:56 PDT 2025
https://github.com/CarlosAlbertoEnciso created https://github.com/llvm/llvm-project/pull/145848
Fixes an incorrect 'REQUIRES' in test case.
The correct value should be:
REQUIRES: webassembly-registered-target
>From 93fa3dec125c861b604b5b8ce6a94af78ed70dfb Mon Sep 17 00:00:00 2001
From: Carlos Alberto Enciso <Carlos.Enciso at sony.com>
Date: Thu, 26 Jun 2025 06:11:16 +0100
Subject: [PATCH] [llvm-debuginfo-analyzer] Fix crash with WebAssembly dead
code (#141616)
https://github.com/llvm/llvm-project/issues/136772
Incorrect handling of 'tombstone' value for WebAssembly.
llvm-debuginfo-analyzer already uses the tombstone approach
to identify dead code. Currently, the tombstone value is
evaluated as std::numeric_limits<uint64_t>::max(). Which is
wrong as it does not take into account the 'Address Byte Size'
from the Compile Unit header.
Fix incorrect 'REQUIRES'. The correct value should be:
REQUIRES: webassembly-registered-target
---
.../llvm-debuginfo-analyzer/WebAssembly/wasm-32bit-tombstone.s | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/llvm/test/tools/llvm-debuginfo-analyzer/WebAssembly/wasm-32bit-tombstone.s b/llvm/test/tools/llvm-debuginfo-analyzer/WebAssembly/wasm-32bit-tombstone.s
index 3e1c1016c9aee..3609b3c8ecc3e 100644
--- a/llvm/test/tools/llvm-debuginfo-analyzer/WebAssembly/wasm-32bit-tombstone.s
+++ b/llvm/test/tools/llvm-debuginfo-analyzer/WebAssembly/wasm-32bit-tombstone.s
@@ -1,4 +1,4 @@
-# REQUIRES: x86-registered-target
+# REQUIRES: webassembly-registered-target
# Test that DWARF tombstones are correctly detected/respected in wasm
# 32 bit object files.
More information about the llvm-commits
mailing list