Diff partially stolen from CheriBSD; these bits need -Wl,-z,notext in order to build in an LLVM world. They are needed for all flavors/sizes of MIPS. This will eventually get fixed in LLVM, but it's unclear when.
Details
Details
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| share/mk/bsd.lib.mk | ||
|---|---|---|
| 286 ↗ | (On Diff #62245) | Maybe rename this to ALLOW_MIPS_SHARED_TEXTREL or something, if it really only applies to mips? |
| 290 ↗ | (On Diff #62245) | I wonder if there is a way to really "fix" this, in the sense that we could get clang and lld to stop attempting to make those relocations? :) |
| 297 ↗ | (On Diff #62245) | Looking at this, I think that you should simply check if ALLOW_SHARED_TEXTREL is defined or not, and stop using the "yes" and "no" values, as this is not a regular MK_FOO knob. Otherwise, you would have to check for: .if !defined(ALLOW_SHARED_TEXTREL) || ${ALLOW_SHARED_TEXTREL} != "yes"instead. |
| 301 ↗ | (On Diff #62245) | Maybe best to decouple this from the -z,notext handling, as people might still want to have linker warnings on mips. |
Comment Actions
I'll address the rest of this later this morning.
| share/mk/bsd.lib.mk | ||
|---|---|---|
| 290 ↗ | (On Diff #62245) | Simon plans to fix it, but he was not able to get me definitive timeline -- which is OK, since he does so much. =) |
Comment Actions
Rename to ALLOW_MIPS_SHARED_TEXTREL and clean up the bsd.lib.mk segment quite a bit. Stop with silly yes/no, check for defined and only do something if mips.
Comment Actions
Looks good. Although I'm not sure if we want this to be conditional on clang/lld since modern GCC+bfd should work (and maybe even clang+new bfd).
Comment Actions
IIRC clang+new bfd still fails as bfd checks object files and catches the relocation in read-only segment. I'll retest that, though, because my memory is dicy here.
Comment Actions
- One last instance of "yes" where the value doesn't matter has been fixed.
- Add enough explanation to make it clear that this won't spontaneously go away due to work in the FreeBSD tree, but rather from clang/lld revisions.