<div dir="ltr">Seems like consistency would be nice (maybe even one day using some common diagnostic handling code - at least for the basic "line/file/column/warning|error/message" syntax). Up to you, though.<br><br>(I'm guessing the compiler would be unlikely to change to add quoting even if it were limited to the subset of cases involving interesting characters - because probably too many people rely on the current syntax... so maybe that's an argument against consistency here)</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 31, 2016 at 11:04 AM Kevin Enderby <<a href="mailto:enderby@apple.com">enderby@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class="gmail_msg">We have found that our IDE often puts object files in directories with names that include things like spaces and punctuation.  So I feel a having the file name in quotes makes sense for object files.  While I understand that is not how the compiler does things.<div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Kev</div></div><div style="word-wrap:break-word" class="gmail_msg"><div class="gmail_msg"><br class="gmail_msg"><div class="gmail_msg"><blockquote type="cite" class="gmail_msg"><div class="gmail_msg">On Oct 31, 2016, at 10:59 AM, David Blaikie via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" class="gmail_msg" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:</div><br class="m_8871371678983633496Apple-interchange-newline gmail_msg"><div class="gmail_msg"><div dir="ltr" class="gmail_msg">FWIW quoting the filename seems a bit odd (at least it's not what the compiler does)</div><br class="gmail_msg"><div class="gmail_quote gmail_msg"><div dir="ltr" class="gmail_msg">On Wed, Oct 26, 2016 at 3:47 PM Justin Bogner via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" class="gmail_msg" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br class="gmail_msg"></div><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: bogner<br class="gmail_msg">
Date: Wed Oct 26 17:37:52 2016<br class="gmail_msg">
New Revision: 285252<br class="gmail_msg">
<br class="gmail_msg">
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=285252&view=rev" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-project?rev=285252&view=rev</a><br class="gmail_msg">
Log:<br class="gmail_msg">
llvm-objdump: Make some error messages more consistent<br class="gmail_msg">
<br class="gmail_msg">
Most of the version of report_error were quoting the filename and<br class="gmail_msg">
printing a colon between the file name and the error message, but this<br class="gmail_msg">
one wasn't doing either of those. Fix the output to be more<br class="gmail_msg">
consistent.<br class="gmail_msg">
<br class="gmail_msg">
Modified:<br class="gmail_msg">
    llvm/trunk/test/Object/macho-invalid.test<br class="gmail_msg">
    llvm/trunk/test/tools/llvm-objdump/malformed-archives.test<br class="gmail_msg">
    llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp<br class="gmail_msg">
<br class="gmail_msg">
Modified: llvm/trunk/test/Object/macho-invalid.test<br class="gmail_msg">
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Object/macho-invalid.test?rev=285252&r1=285251&r2=285252&view=diff" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Object/macho-invalid.test?rev=285252&r1=285251&r2=285252&view=diff</a><br class="gmail_msg">
==============================================================================<br class="gmail_msg">
--- llvm/trunk/test/Object/macho-invalid.test (original)<br class="gmail_msg">
+++ llvm/trunk/test/Object/macho-invalid.test Wed Oct 26 17:37:52 2016<br class="gmail_msg">
@@ -61,14 +61,14 @@ NAME-PAST-EOF-nm-pax: 0000000000000000 6<br class="gmail_msg">
<br class="gmail_msg">
 RUN: not llvm-objdump -t %p/Inputs/macho-bad-archive1.a 2>&1 \<br class="gmail_msg">
 RUN:      | FileCheck -check-prefix NAME-PAST-EOF-ARCHIVE %s<br class="gmail_msg">
-NAME-PAST-EOF-ARCHIVE: macho-bad-archive1.a(macho-invalid-symbol-name-past-eof) truncated or malformed object (bad string index: 4261412866 for symbol at index 0)<br class="gmail_msg">
+NAME-PAST-EOF-ARCHIVE: macho-bad-archive1.a(macho-invalid-symbol-name-past-eof): truncated or malformed object (bad string index: 4261412866 for symbol at index 0)<br class="gmail_msg">
<br class="gmail_msg">
 RUN: not llvm-objdump -macho -arch all -t %p/Inputs/macho-universal-bad1.x86_64.i386 2>&1 \<br class="gmail_msg">
 RUN:      | FileCheck -check-prefix NAME-PAST-EOF-FAT %s<br class="gmail_msg">
-NAME-PAST-EOF-FAT: macho-universal-bad1.x86_64.i386 (for architecture x86_64)  truncated or malformed object (bad string index: 4261412866 for symbol at index 0)<br class="gmail_msg">
+NAME-PAST-EOF-FAT: macho-universal-bad1.x86_64.i386' (for architecture x86_64): truncated or malformed object (bad string index: 4261412866 for symbol at index 0)<br class="gmail_msg">
 RUN: not llvm-objdump -macho -arch all -t %p/Inputs/macho-universal-archive-bad1.x86_64.i386 2>&1 \<br class="gmail_msg">
 RUN:      | FileCheck -check-prefix NAME-PAST-EOF-FAT-ARCHIVE %s<br class="gmail_msg">
-NAME-PAST-EOF-FAT-ARCHIVE: macho-universal-archive-bad1.x86_64.i386(macho-invalid-symbol-name-past-eof) (for architecture x86_64)  truncated or malformed object (bad string index: 4261412866 for symbol at index 0)<br class="gmail_msg">
+NAME-PAST-EOF-FAT-ARCHIVE: macho-universal-archive-bad1.x86_64.i386(macho-invalid-symbol-name-past-eof) (for architecture x86_64): truncated or malformed object (bad string index: 4261412866 for symbol at index 0)<br class="gmail_msg">
<br class="gmail_msg">
 RUN: llvm-nm %p/Inputs/macho-invalid-section-index-getSectionRawName 2>&1 \<br class="gmail_msg">
 RUN:      | FileCheck -check-prefix INVALID-SECTION-IDX-SYMBOL-SEC %s<br class="gmail_msg">
@@ -90,13 +90,13 @@ RUN: not llvm-objdump -macho -private-he<br class="gmail_msg">
 INCOMPLETE-SEGMENT-LOADC: truncated or malformed object (load commands extend past the end of the file)<br class="gmail_msg">
<br class="gmail_msg">
 RUN: not llvm-objdump -macho -private-headers %p/Inputs/macho-bad-archive2.a 2>&1 | FileCheck -check-prefix INCOMPLETE-SEGMENT-LOADC-ARCHIVE %s<br class="gmail_msg">
-INCOMPLETE-SEGMENT-LOADC-ARCHIVE: macho-bad-archive2.a(macho64-invalid-incomplete-segment-load-command) truncated or malformed object (load commands extend past the end of the file)<br class="gmail_msg">
+INCOMPLETE-SEGMENT-LOADC-ARCHIVE: macho-bad-archive2.a(macho64-invalid-incomplete-segment-load-command): truncated or malformed object (load commands extend past the end of the file)<br class="gmail_msg">
<br class="gmail_msg">
 RUN: not llvm-objdump -macho -private-headers -arch all %p/Inputs/macho-universal-bad2.x86_64.i386 2>&1 | FileCheck -check-prefix INCOMPLETE-SEGMENT-LOADC-FAT %s<br class="gmail_msg">
-INCOMPLETE-SEGMENT-LOADC-FAT: macho-universal-bad2.x86_64.i386 (for architecture x86_64) truncated or malformed object (load commands extend past the end of the file)<br class="gmail_msg">
+INCOMPLETE-SEGMENT-LOADC-FAT: macho-universal-bad2.x86_64.i386' (for architecture x86_64): truncated or malformed object (load commands extend past the end of the file)<br class="gmail_msg">
<br class="gmail_msg">
 RUN: not llvm-objdump -macho -private-headers -arch all %p/Inputs/macho-universal-archive-bad2.x86_64.i386 2>&1 | FileCheck -check-prefix INCOMPLETE-SEGMENT-LOADC-FAT-ARCHIVE %s<br class="gmail_msg">
-INCOMPLETE-SEGMENT-LOADC-FAT-ARCHIVE: macho-universal-archive-bad2.x86_64.i386(macho64-invalid-incomplete-segment-load-command) (for architecture x86_64) truncated or malformed object (load commands extend past the end of the file)<br class="gmail_msg">
+INCOMPLETE-SEGMENT-LOADC-FAT-ARCHIVE: macho-universal-archive-bad2.x86_64.i386(macho64-invalid-incomplete-segment-load-command) (for architecture x86_64): truncated or malformed object (load commands extend past the end of the file)<br class="gmail_msg">
<br class="gmail_msg">
 RUN: not llvm-objdump -macho -universal-headers %p/Inputs/macho-invalid-fat 2>&1 | FileCheck -check-prefix INVALID-FAT %s<br class="gmail_msg">
 INVALID-FAT: truncated or malformed fat file (fat_arch_64 structs would extend past the end of the file)<br class="gmail_msg">
@@ -408,4 +408,4 @@ RUN: not llvm-objdump -macho -private-he<br class="gmail_msg">
 INVALID-TWOLEVELHINTS-OFFSET-HNINTS: macho-invalid-twolevelhints-offset-nhints': truncated or malformed object (offset field plus nhints times sizeof(struct twolevel_hint) field of LC_TWOLEVEL_HINTS command 0 extends past the end of the file)<br class="gmail_msg">
<br class="gmail_msg">
 RUN: not llvm-objdump -macho -private-headers %p/Inputs/macho-invalid-fat_cputype 2>&1 | FileCheck -check-prefix INVALID-FAT-CPUTYPE %s<br class="gmail_msg">
-INVALID-FAT-CPUTYPE: macho-invalid-fat_cputype truncated or malformed object (universal header architecture: 0's cputype does not match object file's mach header)<br class="gmail_msg">
+INVALID-FAT-CPUTYPE: macho-invalid-fat_cputype': truncated or malformed object (universal header architecture: 0's cputype does not match object file's mach header)<br class="gmail_msg">
<br class="gmail_msg">
Modified: llvm/trunk/test/tools/llvm-objdump/malformed-archives.test<br class="gmail_msg">
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-objdump/malformed-archives.test?rev=285252&r1=285251&r2=285252&view=diff" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-objdump/malformed-archives.test?rev=285252&r1=285251&r2=285252&view=diff</a><br class="gmail_msg">
==============================================================================<br class="gmail_msg">
--- llvm/trunk/test/tools/llvm-objdump/malformed-archives.test (original)<br class="gmail_msg">
+++ llvm/trunk/test/tools/llvm-objdump/malformed-archives.test Wed Oct 26 17:37:52 2016<br class="gmail_msg">
@@ -47,35 +47,35 @@<br class="gmail_msg">
 # RUN:   %p/Inputs/libbogus8.a \<br class="gmail_msg">
 # RUN:   2>&1 | FileCheck -check-prefix=bogus8 %s<br class="gmail_msg">
<br class="gmail_msg">
-# bogus8: libbogus8.a(???) truncated or malformed archive (long name length: 1234 extends past the end of the member or archive for archive member header at offset 86)<br class="gmail_msg">
+# bogus8: libbogus8.a(???): truncated or malformed archive (long name length: 1234 extends past the end of the member or archive for archive member header at offset 86)<br class="gmail_msg">
<br class="gmail_msg">
 # RUN: not llvm-objdump -s %p/Inputs/libbogus9.a \<br class="gmail_msg">
 # RUN:   2>&1 | FileCheck -check-prefix=bogus9 %s<br class="gmail_msg">
<br class="gmail_msg">
-# bogus9: libbogus9.a(???) truncated or malformed archive (long name offset characters after the '/' are not all decimal numbers: '&a25*' for archive member header at offset 94)<br class="gmail_msg">
+# bogus9: libbogus9.a(???): truncated or malformed archive (long name offset characters after the '/' are not all decimal numbers: '&a25*' for archive member header at offset 94)<br class="gmail_msg">
<br class="gmail_msg">
 # RUN: not llvm-objdump -s %p/Inputs/libbogus10.a \<br class="gmail_msg">
 # RUN:   2>&1 | FileCheck -check-prefix=bogus10 %s<br class="gmail_msg">
<br class="gmail_msg">
-# bogus10: libbogus10.a(???) truncated or malformed archive (long name offset 507 past the end of the string table for archive member header at offset 94)<br class="gmail_msg">
+# bogus10: libbogus10.a(???): truncated or malformed archive (long name offset 507 past the end of the string table for archive member header at offset 94)<br class="gmail_msg">
<br class="gmail_msg">
 # RUN: not llvm-objdump -macho -archive-headers \<br class="gmail_msg">
 # RUN:   %p/Inputs/libbogus11.a \<br class="gmail_msg">
 # RUN:   2>&1 | FileCheck -check-prefix=bogus11 %s<br class="gmail_msg">
<br class="gmail_msg">
-# bogus11: libbogus11.a(hello.c) truncated or malformed archive (characters in UID field in archive header are not all decimal numbers: '~97&' for the archive member header at offset 8)<br class="gmail_msg">
+# bogus11: libbogus11.a(hello.c): truncated or malformed archive (characters in UID field in archive header are not all decimal numbers: '~97&' for the archive member header at offset 8)<br class="gmail_msg">
<br class="gmail_msg">
 # RUN: not llvm-objdump -macho -archive-headers \<br class="gmail_msg">
 # RUN:   %p/Inputs/libbogus12.a \<br class="gmail_msg">
 # RUN:   2>&1 | FileCheck -check-prefix=bogus12 %s<br class="gmail_msg">
<br class="gmail_msg">
-# bogus12: libbogus12.a(hello.c) truncated or malformed archive (characters in GID field in archive header are not all decimal numbers: '#55!' for the archive member header at offset 8)<br class="gmail_msg">
+# bogus12: libbogus12.a(hello.c): truncated or malformed archive (characters in GID field in archive header are not all decimal numbers: '#55!' for the archive member header at offset 8)<br class="gmail_msg">
<br class="gmail_msg">
 # RUN: not llvm-objdump -macho -archive-headers \<br class="gmail_msg">
 # RUN:   %p/Inputs/libbogus13.a \<br class="gmail_msg">
 # RUN:   2>&1 | FileCheck -check-prefix=bogus13 %s<br class="gmail_msg">
<br class="gmail_msg">
-# bogus13: libbogus13.a(hello.c) truncated or malformed archive (characters in AccessMode field in archive header are not all decimal numbers: 'Feed' for the archive member header at offset 8)<br class="gmail_msg">
+# bogus13: libbogus13.a(hello.c): truncated or malformed archive (characters in AccessMode field in archive header are not all decimal numbers: 'Feed' for the archive member header at offset 8)<br class="gmail_msg">
<br class="gmail_msg">
 # RUN: llvm-objdump -macho -archive-headers %p/Inputs/libbogus14.a \<br class="gmail_msg">
 # RUN:   2>&1 | FileCheck -check-prefix=bogus14 %s<br class="gmail_msg">
<br class="gmail_msg">
Modified: llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp<br class="gmail_msg">
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp?rev=285252&r1=285251&r2=285252&view=diff" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp?rev=285252&r1=285251&r2=285252&view=diff</a><br class="gmail_msg">
==============================================================================<br class="gmail_msg">
--- llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp (original)<br class="gmail_msg">
+++ llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp Wed Oct 26 17:37:52 2016<br class="gmail_msg">
@@ -319,14 +319,14 @@ LLVM_ATTRIBUTE_NORETURN void llvm::repor<br class="gmail_msg">
   if (ArchiveName != "")<br class="gmail_msg">
     errs() << ArchiveName << "(" << FileName << ")";<br class="gmail_msg">
   else<br class="gmail_msg">
-    errs() << FileName;<br class="gmail_msg">
+    errs() << "'" << FileName << "'";<br class="gmail_msg">
   if (!ArchitectureName.empty())<br class="gmail_msg">
     errs() << " (for architecture " << ArchitectureName << ")";<br class="gmail_msg">
   std::string Buf;<br class="gmail_msg">
   raw_string_ostream OS(Buf);<br class="gmail_msg">
   logAllUnhandledErrors(std::move(E), OS, "");<br class="gmail_msg">
   OS.flush();<br class="gmail_msg">
-  errs() << " " << Buf;<br class="gmail_msg">
+  errs() << ": " << Buf;<br class="gmail_msg">
   exit(1);<br class="gmail_msg">
 }<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
_______________________________________________<br class="gmail_msg">
llvm-commits mailing list<br class="gmail_msg">
<a href="mailto:llvm-commits@lists.llvm.org" class="gmail_msg" target="_blank">llvm-commits@lists.llvm.org</a><br class="gmail_msg">
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br class="gmail_msg">
</blockquote></div>
_______________________________________________<br class="gmail_msg">llvm-commits mailing list<br class="gmail_msg"><a href="mailto:llvm-commits@lists.llvm.org" class="gmail_msg" target="_blank">llvm-commits@lists.llvm.org</a><br class="gmail_msg"><a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" class="gmail_msg" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br class="gmail_msg"></div></blockquote></div><br class="gmail_msg"></div></div></blockquote></div>