public inbox for dwz@sourceware.org
 help / color / mirror / Atom feed
* [committed][testsuite] Fix partial unit grepping in pr24468.sh
@ 2020-12-20  7:51 Tom de Vries
  0 siblings, 0 replies; only message in thread
From: Tom de Vries @ 2020-12-20  7:51 UTC (permalink / raw)
  To: dwz, jakub

Hi,

I'm running into:
...
FAIL: src/testsuite/dwz.tests/pr24468.sh
...

In more detail, we find the following offsets for partial units:
...
+ offsets='b
6b
da'
...
and then fail to find an import for the one at 6b:
...
++ grep -c 'DW_AT_import.*0x6b' READELF
++ true
+ imports=0
+ '[' 0 -gt 0 ']'
...

But there's actually no partial unit at 6b, the grep matches on a
DW_AT_import:
...
  <6b>  DW_AT_import  : <0xb>  [Abbrev Number: 17 (DW_TAG_partial_unit)]
...

Fix this by filtering out the DW_AT_import lines when grepping for partial
units.

Committed to trunk.

Thanks,
- Tom

[testsuite] Fix partial unit grepping in pr24468.sh

2020-12-20  Tom de Vries  <tdevries@suse.de>

	* testsuite/dwz.tests/pr24468.sh: Fix partial unit grepping.

---
 testsuite/dwz.tests/pr24468.sh | 1 +
 1 file changed, 1 insertion(+)

diff --git a/testsuite/dwz.tests/pr24468.sh b/testsuite/dwz.tests/pr24468.sh
index 4dac9a1..7990e83 100644
--- a/testsuite/dwz.tests/pr24468.sh
+++ b/testsuite/dwz.tests/pr24468.sh
@@ -8,6 +8,7 @@ dwz -m 3 1 2
 readelf -wi 1 > READELF 2>/dev/null
 
 offsets=$(grep '(DW_TAG_partial_unit' READELF \
+	      | grep -v "DW_AT_import" \
 	      | awk '{print $1}' \
 	      | sed 's/.*<//;s/>.*//')
 for off in $offsets; do

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2020-12-20  7:51 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-20  7:51 [committed][testsuite] Fix partial unit grepping in pr24468.sh Tom de Vries

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).