<div dir="ltr">[For patch <a href="http://llvm-reviews.chandlerc.com/D2342">http://llvm-reviews.chandlerc.com/D2342</a>]<div><br></div><div>On the other thread, Bill Wendling wrote:</div><div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
For the original patch, John McCall might be the best person to review it. I believe that's his code originally.</blockquote><div><br></div><div>OK, CC'ing John McCall, although from the version history it does look like you wrote it, Bill.</div>
<div><br></div><div>This change (from 2011) first added the addClause() calls to HandleCallsInBlockInlinedThroughInvoke():</div><div class="gmail_extra"><a href="http://llvm.org/viewvc/llvm-project?view=revision&revision=137576">http://llvm.org/viewvc/llvm-project?view=revision&revision=137576</a><br>
</div></div><div class="gmail_extra"><br></div><div class="gmail_extra">Then this change (from March 2013) added the addClause() calls to forwardResume():</div><div class="gmail_extra"><a href="http://llvm.org/viewvc/llvm-project?view=revision&revision=177680">http://llvm.org/viewvc/llvm-project?view=revision&revision=177680</a></div>
<div class="gmail_extra">-- though this had some de-duplication logic (removed soon after in r177757) which I think would have prevented the duplicated landingpad clauses I'm getting at the moment.</div><div class="gmail_extra">
<br></div><div class="gmail_extra">Cheers,</div><div class="gmail_extra">Mark</div><div class="gmail_extra"><br></div></div>