[PATCH] D19663: [ELF] - Basic support of linkerscript commands: DATA_SEGMENT_ALIGN, DATA_SEGMENT_END, CONSTANT

[Rui Ueyama via llvm-commits]

ruiu added a comment.

This comment is not particularly to this patch, but it seems that you are working on the linker script and starting it from easy part. This patch is, for example, easy to implement. We can add small features such as this later after we complete the main part. There is also a chance that we would end up have to refactor the current code so we would have to rewrite everything -- so adding small features first is a bit risky. Therefore, can you focus on design instead of coding? We need a plan with which we are sure we'll be able to implement all the (well, probably not all, but all sane) linker script features.


http://reviews.llvm.org/D19663