[PATCH] D27506: [ELF] - Print absolute file name in errors when possible.
Rui Ueyama via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 7 08:46:41 PST 2016
ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: ELF/InputFiles.cpp:95
+ std::string Ret = Info.FileName + ":" + std::to_string(Info.Line);
+#ifdef LLVM_ON_WIN32
+ // Converts path to use unix path separators so error message is
----------------
Add a blank line before #ifdef.
================
Comment at: test/ELF/Inputs/undef-debug.s:1
-.file 1 "undef-debug.s"
+.file 1 "folder/undef-debug.s"
.loc 1 3
----------------
nit: "folder" as a term is probably windows-ish too? probably "dir" or something.
https://reviews.llvm.org/D27506
More information about the llvm-commits
mailing list