From 96f00f63b2d65ebe759ae1746c30115e73cbd4f2 Mon Sep 17 00:00:00 2001 From: Michael Buch Date: Thu, 25 Aug 2022 16:00:50 +0100 Subject: [debuginfo-tests] Un-XFAIL no passing unused-merged-value.c test This test would previously expect one of the parameters to have an incorrect DW_AT_location. Stepping through `fun` with a debugger would then no reflect updates to one of the parameters. With a recent change to Clang's DeadArgumentEliminationPass (see `879f5118fc74657e4a5c4eff6810098e1eed75ac`) the generated DWARF does not contain a location for `parama`, and stepping through the function with `lldb` works as expected. Differential Revision: https://reviews.llvm.org/D132664 --- .../debuginfo-tests/dexter-tests/memvars/unused-merged-value.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'cross-project-tests') diff --git a/cross-project-tests/debuginfo-tests/dexter-tests/memvars/unused-merged-value.c b/cross-project-tests/debuginfo-tests/dexter-tests/memvars/unused-merged-value.c index 805062ca17f8..f689aaeb9ada 100644 --- a/cross-project-tests/debuginfo-tests/dexter-tests/memvars/unused-merged-value.c +++ b/cross-project-tests/debuginfo-tests/dexter-tests/memvars/unused-merged-value.c @@ -1,5 +1,7 @@ -// XFAIL: * -// Incorrect location for variable "parama", see PR48719. +// Location for variable "parama" optimized out. +// Previously it would carry incorrect location +// information in debug-info, see PR48719. +// Now, the location is simply not emitted. // REQUIRES: lldb // UNSUPPORTED: system-windows -- cgit v1.2.1