I started to rewrite it in terms of ELFFile. The main challenge nis dealing with the Error object that we don't really care about.<br><br><div class="gmail_quote"><div dir="ltr">On Sat, Nov 10, 2018, 6:05 PM Alexander Shaposhnikov 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">alexshap added inline comments.<br>
<br>
<br>
================<br>
Comment at: llvm/tools/llvm-objcopy/ELF/Note.h:22<br>
+<br>
+static const char *ELF_NOTE_GNU = "GNU";<br>
+<br>
----------------<br>
alexshap wrote:<br>
> mcgrathr wrote:<br>
> > static is always superfluous inside an anonymous namespace.<br>
> > This should be `const char *const` or `constexpr` if LLVM allows newer C++.<br>
> > Except it's unused, so it shouldn't be here at all.<br>
> if i remember correctly the existing Visual Studio build bots were not happy with this, though things might have changed.<br>
(i meant "with this usage of constexpr")<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D54384" rel="noreferrer" target="_blank">https://reviews.llvm.org/D54384</a><br>
<br>
<br>
<br>
</blockquote></div>