[PATCH] D25549: [ELF] - Linkerscript: accept integer values for PHDRS types.

[Rui Ueyama via llvm-commits]

ruiu added inline comments.


================
Comment at: ELF/LinkerScript.cpp:1711-1714
+  unsigned Ret;
+  uint64_t Val;
+  if (readInteger(Tok, Val))
+    Ret = Val;
----------------
Just return Ret here.

  if (readInteger(Tok, Ret))
    return Ret;


================
Comment at: test/ELF/linkerscript/phdrs.s:52
+# RUN: echo "PHDRS {text PT_LOAD FILEHDR PHDRS; \
+# RUN:       foo1 11223344; foo2 0x11223344; foo3 0x112233440000; } \
+# RUN:       SECTIONS { . = SIZEOF_HEADERS; .foo : { *(.*) } : text : foo1}" > %t1.script
----------------
I don't see a reason to have three test cases here. One pattern should suffice.


https://reviews.llvm.org/D25549