Don't wait on me until June 10th. I'm just skimming emails. <br><br><div class="gmail_quote"><div dir="ltr">On Thu, May 31, 2018, 4:09 PM James Henderson via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">jhenderson accepted this revision.<br>
jhenderson added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM with the inline comments, but please wait for @alexshap or @jakehehrlich to confirm they're happy.<br>
<br>
<br>
<br>
================<br>
Comment at: test/tools/llvm-objcopy/null-symbol.test:21<br>
+#CHECK-NEXT:  Symbol {<br>
+#CHECK-NEXT:    Name:{{[ ]*}}(0)<br>
+#CHECK-NEXT:    Value: 0x0<br>
----------------<br>
I think this can be simplified to `Name: (0)`. Whitespace (except new lines) is not significant for FileCheck, except to divide other non-whitespace, i.e. `Name:    (0)` (many spaces) is identical to `Name: (0)` (one space) but not `Name:(0)` (no spaces).<br>
<br>
<br>
================<br>
Comment at: tools/llvm-objcopy/Object.cpp:203-205<br>
+  for (auto Sym = std::begin(Symbols) + 1, E = std::end(Symbols); Sym != E;<br>
+       ++Sym)<br>
+    Callable(**Sym);<br>
----------------<br>
Hmm... now that I think about it, could we use std::for_each here?<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D47414" rel="noreferrer" target="_blank">https://reviews.llvm.org/D47414</a><br>
<br>
<br>
<br>
</blockquote></div>