Skip to content

Conversation

@aokblast
Copy link
Contributor

This patch is to reflect the change in supporting more than PN_XNUM segments in readobj.

@llvmbot
Copy link
Member

llvmbot commented Oct 27, 2025

@llvm/pr-subscribers-llvm-binary-utilities

Author: None (aokblast)

Changes

This patch is to reflect the change in supporting more than PN_XNUM segments in readobj.


Full diff: https://github.com/llvm/llvm-project/pull/165279.diff

1 Files Affected:

  • (modified) llvm/test/tools/llvm-objcopy/ELF/many-sections.test (+2-2)
diff --git a/llvm/test/tools/llvm-objcopy/ELF/many-sections.test b/llvm/test/tools/llvm-objcopy/ELF/many-sections.test
index 6622db237026f..4c618acb5c951 100644
--- a/llvm/test/tools/llvm-objcopy/ELF/many-sections.test
+++ b/llvm/test/tools/llvm-objcopy/ELF/many-sections.test
@@ -6,8 +6,8 @@ RUN: llvm-readobj --file-headers --sections --symbols %t2 | FileCheck %s
 RUN: llvm-readelf --symbols %t2 | FileCheck --check-prefix=SYMS %s
 
 ## The ELF header should have e_shnum == 0 and e_shstrndx == SHN_XINDEX.
-# CHECK:        SectionHeaderCount: 0
-# CHECK-NEXT:   StringTableSectionIndex: 65535
+# CHECK:        SectionHeaderCount: 0 (65540)
+# CHECK-NEXT:   StringTableSectionIndex: 65535 (65539)
 
 ## The first section header should store the real section header count and
 ## shstrndx in its fields.

@aokblast aokblast force-pushed the users/aokblast/readelf/pxnum_support branch from 1dc7fc0 to fe99baf Compare October 30, 2025 16:46
@aokblast aokblast force-pushed the users/aokblast/readobj/fix_tc_with_pnxnum branch from 605743c to 214170a Compare October 30, 2025 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants