[PATCH] [ELF] Implement --stats

Rui Ueyama ruiu at google.com
Thu Feb 19 11:55:12 PST 2015


LGTM


REPOSITORY
  rL LLVM

================
Comment at: lib/Driver/GnuLdDriver.cpp:181
@@ +180,3 @@
+    diag << "total time in link " << t.getProcessTime() << "\n";
+    diag << "data size " << t.getMemUsed() << "\n";
+  }
----------------
total -> Total

"data size" may be confusing -- this can be read as the total number of bytes read from disk, for example. Needs a bit of wordsmithing.

http://reviews.llvm.org/D7657

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list