<div dir="ltr"><div>This is really a question for John, but I have a feeling that he'll be happy to see this code go.<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 30, 2014 at 8:56 AM, Logan Chien <span dir="ltr"><<a href="mailto:tzuhsiang.chien@gmail.com" target="_blank">tzuhsiang.chien@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Hi,<br><br></div><div>I would like to propose to remove the CleanupHackLevel from CGException.cpp.  This hack was originally written for the old exception handling intrinsics, which have several issues with function inliner.  (Read [1] for more background on this hack.)<br>


<br>Since LLVM 3.0, the new exception handling mechanism has been proposed and implemented [1].  The landingpad and resume instructions are added to LLVM IR.  AFAIK, the new exception handling mechanism works very well, and solved most of the problems.<br>


</div><br>From the svn log, the CleanupHackLevel code hasn't be used for a long time.  It is guarded by a compile-time constant `CleanupHackLevel`.  The alternative cases will be considered as the dead code and removed by the compiler.  In addition, there is no test case covering these cases.  I feel that it's a good time to remove them.<br>


<br></div><div>I have attached the patch to remove the code.  Please have a look.  Feel free to let me know if you have any suggestions.  Thanks.<br><br></div><div>Sincerely,<br></div><div>Logan<br></div><div><br>
[1] <a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2011-July/041748.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvmdev/2011-July/041748.html</a><br></div></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>