[PATCH] D36104: [AArch64] Coalesce Copy Zero during instruction selection

Haicheng Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 1 19:32:01 PDT 2017


haicheng added inline comments.


================
Comment at: test/CodeGen/AArch64/arm64-addr-type-promotion.ll:31
 ; CHECK-NEXT: ldrb [[LOADEDVAL4:w[0-9]+]], {{\[}}[[BLOCKBASE2]], #2]
+; CHECK-NEXT: mov w0, wzr
 ; CHECK-NEXT: cmp [[LOADEDVAL3]], [[LOADEDVAL4]]
----------------
gberry wrote:
> Is this a regression?
This corresponds to block if.end25.  Do you think it is better to create a new block to sink mov wzr here?  I think neither sinking or not is a clear win since it depends on which path the control flow takes.  Also, the cost of mov wzr can be as cheap as zero.   So, I am bias to keep the current CFG.


Repository:
  rL LLVM

https://reviews.llvm.org/D36104





More information about the llvm-commits mailing list