<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><a class="gmail_plusreply" id="m_-8430654449467097977plusReplyChip-0" href="mailto:rahmanl@google.com" target="_blank">+Rahman Lavaee</a> who is working on exception support for basicblock sections. <br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 10, 2020 at 4:29 PM Modi Mo <<a href="mailto:modimo@fb.com" target="_blank">modimo@fb.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">





<div lang="EN-US">
<div>
<p style="margin:0in;text-align:justify"><span style="font-family:Arial,sans-serif;color:black">>Exceptions</span><u></u><u></u></p>
<p style="margin:0in;text-align:justify"><span style="font-family:Arial,sans-serif;color:black">>All eh pads are grouped together regardless of their coldness and are part of the original function. There are outstanding issues with splitting eh pads if they
 reside in separate sections in the binary. This remains as part of future work.<u></u><u></u></span></p>
<p style="margin:0in;text-align:justify"><span style="font-family:Arial,sans-serif;color:black"><u></u> <u></u></span></p>
<p style="margin:0in;text-align:justify">Can you elaborate more on the outstanding issues with splitting eh pads?<u></u><u></u></p>
<p style="margin:0in;text-align:justify"><u></u> <u></u></p>
<p style="margin:0in;text-align:justify">From my dip into the unwind map in gcc_except_table the current encoding to landing pads using function-start relative offsets doesn’t extend well to moving the pads somewhere else. Are there additional issues with splitting
 specific pads out?</p></div></div></blockquote><div><span class="gmail_default" style="font-size:small">That's correct, we have a pending patch to add support for this. </span><a href="https://reviews.llvm.org/D73739" target="_blank">https://reviews.llvm.org/D73739</a><span class="gmail_default"></span></div><div><span class="gmail_default">IIUC with this patch, we can treat a set of ehpads as hot or cold and move them around as needed.</span></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div lang="EN-US"><div><p style="margin:0in;text-align:justify"><u></u><u></u></p>
<p style="margin:0in;text-align:justify"><u></u> <u></u></p>
<p style="margin:0in;text-align:justify">Best,<u></u><u></u></p>
<p style="margin:0in;text-align:justify">Modi</p></div></div></blockquote><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div lang="EN-US"><div><div><div>
</div>
</div>
</div>
</div>

</blockquote></div></div>