<div dir="auto">+1<div dir="auto"><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, May 30, 2018, 10:45 PM Jake Ehrlich via Phabricator via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">jakehehrlich added a subscriber: alexshap.<br>
jakehehrlich added a comment.<br>
<br>
Quick drop in. What I was thinking about before was to just never pass the<br>
null symbol to remove or anything else. It can still be there it just<br>
shouldn't be allowed to be updated or removed.<br>
<br>
As for how the null section header should be treated we have to consider<br>
large indexes before proceeding.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D47414" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D47414</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank" rel="noreferrer">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>