================ @@ -3840,14 +3849,15 @@ void GNUELFDumper<ELFT>::printRelRelaReloc(const Relocation<ELFT> &R, template <class ELFT> static void printRelocHeaderFields(formatted_raw_ostream &OS, unsigned SType, - const typename ELFT::Ehdr &EHeader) { + const typename ELFT::Ehdr &EHeader, + uint64_t CrelHdr = 0) { bool IsRela = SType == ELF::SHT_RELA || SType == ELF::SHT_ANDROID_RELA; if (ELFT::Is64Bits) OS << " Offset Info Type Symbol's " "Value Symbol's Name"; else OS << " Offset Info Type Sym. Value Symbol's Name"; - if (IsRela) + if (IsRela || (SType == ELF::SHT_CREL && (CrelHdr & CREL_HDR_ADDEND))) ---------------- jh7370 wrote:
Could we rename the `IsRela` to `HasAddend` or similar, and then add the CREL logic to the initialisation, since `IsRela` isn't used for anything else. Alternatively, we could simply inline the variable here. https://github.com/llvm/llvm-project/pull/91280 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits