[PATCH] D21995: [ELF] Implement minimal PHDRS parser and section-to-segment assignment

[George Rimar via llvm-commits]

grimar added inline comments.

================
Comment at: ELF/LinkerScript.cpp:286
@@ +285,3 @@
+  ssize_t RelroNum = -1;
+  Phdr *Load = nullptr;
+  uintX_t Flags = PF_R;
----------------
evgeny777 wrote:
> Well, ErrorOr<T> looks a way too heavy to represent valid/invalid array index, doesn't it?
> 
Right. That is why I suggested to use int32_t/uint32_t depending on what you need here. It is what we generally use in lld for such cases.
But we do not use int and never used ssize_t yet I think.


http://reviews.llvm.org/D21995