[PATCH] D56060: ELF: BLOCK Keyword linker script support

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 24 01:58:17 PST 2018


grimar added a comment.

In according to the documentation, `BLOCK` is a synonym for `ALIGN` for compatibility with the older linker scripts. Given that its name sounds meaningless/confusing,
I would prefer to see the `ALIGN` in the scripts. Hence I suggest not to add this to LLD but fix the script. How does it sound?


Repository:
  rLLD LLVM Linker

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56060/new/

https://reviews.llvm.org/D56060





More information about the llvm-commits mailing list