<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Hi all,<div class=""><br class=""></div><div class="">I would like to start another discussion on bug 23991 (<a href="https://bugs.llvm.org/show_bug.cgi?id=23991" class="">https://bugs.llvm.org/show_bug.cgi?id=23991</a>), regarding the missing reregistration of EH frames in ORCv2.</div><div class=""><br class=""></div><div class="">Compared to the time where the bugzilla thread was created, I think the structure of ORCv2 is fairly stable now. So I try to create a small patch trying to fix that: <a href="https://reviews.llvm.org/D71683" class="">https://reviews.llvm.org/D71683</a></div><div class=""><br class=""></div><div class="">My concern is that whether this is the proper place to put the deregistration logic. And whether we should put a note to remind future MemoryManager implementations to take care of the deregistration since it’s not suitable to put <font color="#000000" class=""><span style="caret-color: rgb(0, 0, 0);" class="">deregistration in MemoryManager’s dtor as explained in the code review description.</span></font></div><div class=""><font color="#000000" class=""><span style="caret-color: rgb(0, 0, 0);" class=""><br class=""></span></font></div><div class=""><font color="#000000" class=""><span style="caret-color: rgb(0, 0, 0);" class="">B.R.</span></font></div><div class=""><font color="#000000" class=""><span style="caret-color: rgb(0, 0, 0);" class="">Min-Yih</span></font></div></body></html>