[PATCH] D40652: [ELF] - Produce relocation section name consistent with output section name when --emit-reloc used with linker script.

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 30 15:02:38 PST 2017


Rui Ueyama via Phabricator <reviews at reviews.llvm.org> writes:

> ruiu added inline comments.
>
>
> ================
> Comment at: ELF/Writer.cpp:91
> +StringRef elf::getOutputSectionName(InputSectionBase *S) {
> +  StringRef Name = S->Name;
> +
> ----------------
> This saves only two characters. Just use S->Name instead of Name.

I agree, but please do that as a followup as otherwise the patch is much
harder to read.

Cheers,
Rafael


More information about the llvm-commits mailing list