[llvm] r240655 - llvm-nm: print 'n' instead of '?'

Rafael Espindola rafael.espindola at gmail.com
Thu Jun 25 09:01:54 PDT 2015


Author: rafael
Date: Thu Jun 25 11:01:53 2015
New Revision: 240655

URL: http://llvm.org/viewvc/llvm-project?rev=240655&view=rev
Log:
llvm-nm: print 'n' instead of '?'

This matches gnu nm and has the advantage that there is a upper case N.

Modified:
    llvm/trunk/test/Object/X86/nm-print-size.s
    llvm/trunk/tools/llvm-nm/llvm-nm.cpp

Modified: llvm/trunk/test/Object/X86/nm-print-size.s
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Object/X86/nm-print-size.s?rev=240655&r1=240654&r2=240655&view=diff
==============================================================================
--- llvm/trunk/test/Object/X86/nm-print-size.s (original)
+++ llvm/trunk/test/Object/X86/nm-print-size.s Thu Jun 25 11:01:53 2015
@@ -1,7 +1,12 @@
 // RUN: llvm-mc %s -o %t -filetype=obj -triple=x86_64-pc-linux
 // RUN: llvm-nm --print-size %t | FileCheck %s
 
-// CHECK: 0000000000000000 ffffffffffffffff t a
+// CHECK: 0000000000000000 ffffffffffffffff n a
+// CHECK: 0000000000000000 0000000000000000 N b
 
+        .section foo
 a:
         .size a, 0xffffffffffffffff
+
+        .global b
+b:

Modified: llvm/trunk/tools/llvm-nm/llvm-nm.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-nm/llvm-nm.cpp?rev=240655&r1=240654&r2=240655&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-nm/llvm-nm.cpp (original)
+++ llvm/trunk/tools/llvm-nm/llvm-nm.cpp Thu Jun 25 11:01:53 2015
@@ -675,7 +675,7 @@ static char getSymbolNMTypeChar(ELFObjec
         .Default('?');
   }
 
-  return '?';
+  return 'n';
 }
 
 static char getSymbolNMTypeChar(COFFObjectFile &Obj, symbol_iterator I) {





More information about the llvm-commits mailing list