diff options
| author | David Lamparter <equinox@opensourcerouting.org> | 2023-01-06 16:59:55 +0100 |
|---|---|---|
| committer | David Lamparter <equinox@opensourcerouting.org> | 2023-01-06 17:05:16 +0100 |
| commit | ee4a6b9f654fbd4e07485037481ad6b4766bbc6d (patch) | |
| tree | 3dafd83ce739957b637c25beff7077955a8c530c /lib/xref.h | |
| parent | zebra: do not load/store wider-than-ptr atomics (diff) | |
| download | frr-ee4a6b9f654fbd4e07485037481ad6b4766bbc6d.tar.xz frr-ee4a6b9f654fbd4e07485037481ad6b4766bbc6d.zip | |
lib: disable xref ELF note on mips64el
mips64el does not have a 64-bit PC-relative relocation, which is needed
to emit the ELF note for xrefs. Disabling the ELF note means clippy
takes the fallback path using section headers, so everything does still
work (... unless you strip the section headers.)
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
Diffstat (limited to 'lib/xref.h')
| -rw-r--r-- | lib/xref.h | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/lib/xref.h b/lib/xref.h index 0e3f00f69..37242bd79 100644 --- a/lib/xref.h +++ b/lib/xref.h @@ -208,8 +208,19 @@ extern const struct xref * const __stop_xref_array[1] DSO_LOCAL; * some build issue with it just add -DFRR_XREF_NO_NOTE to your build flags * to disable it. */ -#ifdef FRR_XREF_NO_NOTE +#if defined(FRR_XREF_NO_NOTE) || defined(__mips64) #define XREF_NOTE "" + +/* mips64 note: MIPS64 (regardless of endianness, both mips64 & mips64el) + * does not have a 64-bit PC-relative relocation type. Unfortunately, a + * 64-bit PC-relative relocation is exactly what the below asm magic emits. + * Therefore, the xref ELF note is permanently disabled on MIPS64. + * + * For some context, refer to https://reviews.llvm.org/D80390 + * + * As noted above, xref extraction still works through the section header + * path, so no functionality is lost. + */ #else #if __SIZEOF_POINTER__ == 4 |