Something like this? Do I need to check if the MachineFunction has a valid Function the way MachineBasicBlock checks and returns "(null)"? None of the callers I replaced seemed to be checking for a valid result from getFunction().<br>
<br><div class="gmail_quote">On Tue, Aug 21, 2012 at 5:06 PM, Craig Topper <span dir="ltr"><<a href="mailto:craig.topper@gmail.com" target="_blank">craig.topper@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'll work on this tonight.<div class="HOEnZb"><div class="h5"><br><br><div class="gmail_quote">On Tue, Aug 21, 2012 at 1:25 PM, Jakob Stoklund Olesen <span dir="ltr"><<a href="mailto:stoklund@2pi.dk" target="_blank">stoklund@2pi.dk</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><br>
On Aug 21, 2012, at 1:14 PM, Craig Topper <<a href="mailto:craig.topper@gmail.com" target="_blank">craig.topper@gmail.com</a>> wrote:<br>
<br>
> I think the other register allocators have the same cast and should be fixed too.<br>
<br>
</div>Even better, add MachineFunction::getName(), and stop including "llvm/Function.h" in every backend pass.<br>
<br>
See MachineBasicBlock::getName().<br>
<span><font color="#888888"><br>
/jakob<br>
<br>
</font></span></blockquote></div><br><br clear="all"><br></div></div><span class="HOEnZb"><font color="#888888">-- <br>~Craig<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>~Craig<br>