[PATCH] D27415: [ELF] - Replace MergeOutputSection with synthetic input section MergeSection.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 31 02:09:47 PST 2017


>I have spent some time simplifying the patch. It is attached and I
>think it is substantially simpler than the previous version.
>

Thanks  a for your time, your version looks better.
Will you commit then ?

>It can be made simpler still by removing a few hacks to set EntSize
>and SHF_MERGE on the output flags. I suggest making that in a followup
>patch so as to not change too many tests at once.

Yes, I also set EntSize in my version to avoid changing testcase in currect patch.
I can prepare such patch after landing this.

>This patch should also enable some simplifications in the linker script code.
>
>George, would this work for you? I included a simplified version of
>the testcase you mentioned.
>
>Cheers,
>Rafael

Seems works perfectly, I checked reproduce I had for videocard symbols and they have different values now.
Adds a big chance for kernel to boot I suppose :) 

Thanks again.

George.


More information about the llvm-commits mailing list