This fixes https://bugs.llvm.org/show_bug.cgi?id=41305. GNU objcopy --set-section-flags/--rename-section flags are case-insensitive, so this patch updates llvm-objcopy to match.
Details
Details
- Reviewers
grimar rupprecht jakehehrlich • espindola alexander-shaposhnikov - Commits
- rGf5b181e16db7: [NFC] Address missed review comment for test
rL357594: [NFC] Address missed review comment for test
rGd931cf3e46fd: [llvm-objcopy] Make section rename/set flags case-insensitive
rL357590: [llvm-objcopy] Make section rename/set flags case-insensitive
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
LGTM with a nit.
test/tools/llvm-objcopy/ELF/set-section-flags.test | ||
---|---|---|
69 ↗ | (On Diff #193487) | Perhaps separating into two independent blocks would be a bit more readable? # Upper case flags: # RUN: ... ... # Mixed-case flags: # RUN: ... ... |