[PATCH] D22749: [ELF] - Linkerscript: Implemented SORT command

[George Rimar via llvm-commits]

grimar added a comment.

In https://reviews.llvm.org/D22749#496533, @emaste wrote:

> > I don't know yet, but we need to do something. It's probably nice to slow down a bit and make the code simpler and easier to read.
>
>
> One little comment, we're now quite close to fully handling the FreeBSD kernel linker script.
>
> What's missing:
>
> 1. AT (https://reviews.llvm.org/D19272)
> 2. SORT (https://reviews.llvm.org/D22749)
> 3. Filename specification (e.g. `KEEP (*crtbegin.o(.ctors))`)
> 4. EXCLUDE_FILE (https://reviews.llvm.org/D22795)
> 5. CONSTRUCTORS
> 6. `. =` assignment in output section (PR28720)
>
>   I think `CONSTRUCTORS` is just a nop for ELF (and we can delete it from the linker script) and PR28720 can also be addressed easily in FreeBSD's linker script instead of changing lld.
>
>   At least as far as the FreeBSD base system and kernel is concerned there's not much left to implement, regardless of whether we pause now to refactor and clean up or not.


Thanks for summarizing this, Ed !

5. CONSTRUCTOR - It is https://reviews.llvm.org/D22848


https://reviews.llvm.org/D22749