[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?
More information about the llvm-commits