[PATCH] D16924: [ELF] - Linker script expressions, ASSERT() command.

[Rui Ueyama via llvm-commits]

ruiu added a comment.

Hi George,

This is probably too much at this moment. I'm not sure if we want to support ASSERT, but even if we want it, it is something we could add as a final step to complete the linker script support (until that point, we will have everything, so adding ASSERT would be very easy.) The expressions in ASSERT is the features users would use in other contexts. Because that is not plugged in to anywhere in your patch, your patch doesn't demonstrate that that would work for a minimal use case.

If you want to start working on the linker script support, can you start from writing a proposal about the overall design? It needs to be simple, easy to implement, and fast. I have a few ideas for that, but I don't have enough time to think deeply about them. Feel free to discuss your idea with me.


http://reviews.llvm.org/D16924