[llvm-commits] [llvm] r65996 - in /llvm/trunk: lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp lib/Target/X86/X86RegisterInfo.td test/CodeGen/X86/pr3701.ll

Evan Cheng echeng at apple.com
Tue Mar 3 18:48:21 PST 2009


On Mar 3, 2009, at 6:00 PM, Dan Gohman wrote:

>
> On Mar 3, 2009, at 5:41 PM, Evan Cheng wrote:
>
>> Author: evancheng
>> Date: Tue Mar  3 19:41:49 2009
>> New Revision: 65996
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=65996&view=rev
>> Log:
>> Fix PR3701. 1. X86 target renamed eflags register to flags. This
>> matches what llvm-gcc generates so codegen knows flags register is
>> being clobbered by inline asm. 2. BURR scheduler should also check
>> if inline asm nodes can clobber "live" physical registers.
>> Previously it was only checking target nodes with implicit defs.
>
> What about the "~{dirflag}" part? Since DF isn't modeled
> separately, should "dirflag" be mapped to "flags"?

Registers are matched by name. I don't see a need to special case this  
since all 3 flag registers are added to inline asm instructions.

Evan

>
> Dan
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list