[PATCH] D27712: [ELF] - Do not crash when move location counter backward.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 13 09:24:53 PST 2016


grimar added inline comments.


================
Comment at: ELF/LinkerScript.cpp:467-469
+      uintX_t Val = AssignCmd->Expression(Dot);
+      if (Val < Dot)
+        error("unable to move location counter backward for: " + CurOutSec->Name);
----------------
ruiu wrote:
> If `Dot` overflows towards negative, your expression cannot detect the error because `Val` would be a very large number. Can you handle this case as well?
I'll update the patch tomorrow for that case.


https://reviews.llvm.org/D27712





More information about the llvm-commits mailing list