<div dir="ltr">This LGTM; I don't see a clear owner for this part of the code, and most of it seems to have been written a long time ago by Chris, so hopefully if anyone else has opinions they will speak up soon.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 17, 2013 at 6:39 PM, Mark Seaborn <span dir="ltr"><<a href="mailto:mseaborn@chromium.org" target="_blank">mseaborn@chromium.org</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">Fix DeleteDeadVarargs not to crash on functions referenced by BlockAddresses<br><br>This pass was assuming that if hasAddressTaken() returns false for a<br>
function, the function's only uses are call sites.  That's not true<br>
because there can be references by BlockAddresses too.<br><br>Fix the pass to handle this case.  Fix<br>BlockAddress::replaceUsesOfWithOnConstant() to allow a function's type<br>to be changed by RAUW'ing the function with a bitcast of the recreated<br>

function.<br><br></div>
</blockquote></div><br></div>