[PATCH] D37013: [ELF] - Add additional information about location when emiting linkerscript's parsing errors.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 23 08:29:11 PDT 2017


grimar added inline comments.


================
Comment at: ELF/ScriptParser.cpp:608
+  ErrorPrefix = "unable to parse description for output section '" +
+                OutSec.str() + "':\n>>> ";
   OutputSection *Cmd =
----------------
ruiu wrote:
> Don't include ">>>" in a prefix. Just end it with ": ".
I did that because message printed was huge for me:

`error: {{.*}}.script.inc:4: unable to parse description for output section 'boom': malformed number: .temp`

where `{{.*}}.script` was a long path itself.


https://reviews.llvm.org/D37013





More information about the llvm-commits mailing list