<div class="gmail_quote">On Fri, May 14, 2010 at 3:02 PM, Jakob Stoklund Olesen <span dir="ltr"><<a href="mailto:stoklund@2pi.dk">stoklund@2pi.dk</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

==============================================================================<br>
--- llvm/trunk/lib/CodeGen/RegAllocFast.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/RegAllocFast.cpp Fri May 14 17:02:56 2010<br>
@@ -394,7 +394,8 @@<br>
<br>
   // Ignore invalid hints.<br>
   if (Hint && (!TargetRegisterInfo::isPhysicalRegister(Hint) ||<br>
-               !RC->contains(Hint) || UsedInInstr.test(Hint)))<br>
+               !RC->contains(Hint) || UsedInInstr.test(Hint)) ||<br>
+               !Allocatable.test(Hint))<br></blockquote><div><br></div><div>I think the !Allocatable.test(Hint) is intended to be in the group of ||s? Every other use groups in that way, the indentation agrees, and a GCC warning pointed this out, so I went ahead and switch it in r103869. Please review, and I'll fix if this was intentional (and you don't beat me to it!).</div>
</div>