Page MenuHomeFreeBSD
Differential D18808

fix rtld-elf compilation warning for powerpc64 ELFv2 ABI
ClosedPublic
Actions

Authored by alfredo on Jan 10 2019, 12:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 23, 8:34 PM
Unknown Object (File)
May 4 2024, 3:20 PM
Unknown Object (File)
Apr 26 2024, 8:17 AM
Unknown Object (File)
Apr 20 2024, 2:09 PM
Unknown Object (File)
Apr 19 2024, 4:37 PM
Unknown Object (File)
Apr 19 2024, 4:33 PM
Unknown Object (File)
Apr 19 2024, 4:33 PM
Unknown Object (File)
Apr 19 2024, 4:33 PM
View All 41 Files
Subscribers
bdragon
imp
luporl
Contributor Reviews (src)

Details

Reviewers
kib
bdragon
emaste
jhibbits
Commits
rS342985: Fix rtld-elf compilation warning for powerpc64 ELFv2 ABI
Summary

reloc_jmpslot function parameter 'defobj' is not used when using ELFv2 ABI

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 21912
Build 21156: arc lint + arc unit

Event Timeline

alfredo created this revision.Jan 10 2019, 12:05 PM
Herald added subscribers: Contributor Reviews (src), bdragon, imp. · View Herald TranscriptJan 10 2019, 12:05 PM
Harbormaster completed remote builds in B21912: Diff 52730.Jan 10 2019, 12:05 PM
alfredo edited the summary of this revision. (Show Details)Jan 10 2019, 12:23 PM
alfredo added reviewers: kib, bdragon.
kib added inline comments.Jan 10 2019, 12:30 PM
libexec/rtld-elf/powerpc64/reloc.c
450

You can add __unused attribute to the argument. It is innocent when not true.

luporl added a subscriber: luporl.Jan 10 2019, 2:50 PM
alfredo added reviewers: emaste, jhibbits.Jan 10 2019, 3:11 PM
alfredo updated this revision to Diff 52742.Jan 10 2019, 3:15 PM

as per review, added __unused attributed instead

Harbormaster completed remote builds in B21917: Diff 52742.Jan 10 2019, 3:15 PM
emaste accepted this revision.Jan 10 2019, 3:22 PM
This revision is now accepted and ready to land.Jan 10 2019, 3:22 PM
jhibbits accepted this revision.Jan 10 2019, 3:28 PM
kib accepted this revision.Jan 10 2019, 3:28 PM
bdragon accepted this revision.Jan 10 2019, 4:14 PM

OK, I'm fine with this solution.

bdragon mentioned this in D18565: Fix warning in PowerPC64 ELFv2 build of rtld-elf..Jan 10 2019, 4:17 PM
Closed by commit rS342985: Fix rtld-elf compilation warning for powerpc64 ELFv2 ABI (authored by jhibbits). · Explain WhyJan 13 2019, 2:33 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
libexec/
rtld-elf/
powerpc64/
1 line

Diff 52730

View Options

libexec/rtld-elf/powerpc64/reloc.c

Loading...