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

[Chandler Carruth]

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/6c92841b/attachment.html>