[llvm-commits] Ready for commit: X86 load/dec-or-store/store to dec-or-inc

[Joel Jones]

Yea, but I'm having trouble authenticating to the server.  Chris and I are working on it.

Joel

On Mar 28, 2012, at 4:33 PM, Chandler Carruth wrote:

> On Wed, Mar 28, 2012 at 2:16 PM, Joel Jones <joel_k_jones at apple.com> wrote:
> I only have "commit after approval permission", so if someone could do the honors.
> 
> I think Evan's comment of "these look good" (modulo his merge request) was approval. =]
>  
> 
> The second attachment is after doing:
> 
> svn rename dec-eflags-lower.ll rd-mod-wr-eflag
> 
> in test/CodeGen/X86.  Below my name is the check-in comment.
> 
> Joel
> 
> For X86, change load/dec-or-inc/store into dec-or-inc, respectively.
> 
> This is a code change to add support for changing instruction sequences of the form:
> 
>  load
>  inc/dec of 8/16/32/64 bits
>  store
> 
> into the appropriate X86 inc/dec through memory instruction:
> 
>  inc[qlwb] / dec[qlwb]
> 
> The checks that were in X86DAGToDAGISel::Select(SDNode *Node)>>ISD::STORE have been extracted to isLoadIncOrDecStore and reworked to use the better
> named wrappers for getOperand(unsigned) (e.g. getOffset()) and replaced Chain.getNode() with LoadNode.  The comments have also been expanded.
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120328/d501a289/attachment.html>