[llvm-commits] [patch] Fix pr11415

Jakob Stoklund Olesen stoklund at 2pi.dk
Mon Nov 21 21:12:14 PST 2011


On Nov 21, 2011, at 8:27 PM, Rafael Espíndola wrote:

> It looks like the fast register allocator thinks that early clobbers
> are defined a bit earlier than they really are.
> 
> The problem is that when a register is both a tied use and an early
> clobber of a asm statement, we process the early clobber first and
> then think that the register is already set, which causes the missing
> copy to RBP in llvm.org/pr11415.
> 
> This is one case I am not sure how to test. I can include the testcase
> that is attached to the PR, but it is fairly brittle.

The patch looks good. I am not sure why you can't test it. Isn't it enough with a "&=r" inline asm constraint?

/jakob





More information about the llvm-commits mailing list