[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