[PATCH] [AArch64] Check Dest Resgister Liveness in CondOpt pass

Z. Zheng zhaoshiz at codeaurora.org
Thu Oct 30 16:56:46 PDT 2014


Hi apazos, t.p.northover, Jiangning, sdmitrouk,

  CondOpt pass does not check for destination register liveness of subs (alias with cmp) instructions.

  Our internal test reveals such case should not be transformed:

    cmp x17, #3
    b.lt .LBB10_15
    ...
    subs x12, x12, #1
    b.gt .LBB10_1

  where x12 is a liveout, becomes:

    cmp x17, #2
    b.le .LBB10_15
    ...
    subs x12, x12, #2
    b.ge .LBB10_1

http://reviews.llvm.org/D6048

Files:
  lib/Target/AArch64/AArch64ConditionOptimizer.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6048.15594.patch
Type: text/x-patch
Size: 1212 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141030/29edb4c2/attachment.bin>


More information about the llvm-commits mailing list