Quick drop in. What I was thinking about before was to just never pass the null symbol to remove or anything else. It can still be there it just shouldn't be allowed to be updated or removed.<div><br></div><div>As for how the null section header should be treated we have to consider large indexes before proceeding.</div><div><br><div class="gmail_quote"><div dir="ltr">On Wed, May 30, 2018, 5:59 PM Paul Semel 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">paulsemel added a comment.<br>
<br>
Ok, I'm getting back to you because I really think that we want to go forward on this one, as this fixes a really "big" bug.<br>
<br>
Are you guys totally convinced that we might keep this symbol ?<br>
<br>
If so, I will change the implementation, even though I truly think not exposing the symbol will simplify the future development of `llvm-objcopy`.<br>
<br>
cc: @jhenderson @alexshap<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></div>