[PATCH] D38146: [AArch64] Fix bug in store of vector 0 DAGCombine.

Geoff Berry via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 21 11:54:47 PDT 2017


gberry added a comment.

Thanks Eli.



================
Comment at: lib/Target/AArch64/AArch64ISelLowering.cpp:9463
+  }
+  SDValue SplatVal = DAG.getCopyFromReg(St.getChain(), DL, ZeroReg, ZeroVT);
   return splitStoreSplat(DAG, St, SplatVal, NumVecElts);
----------------
efriedma wrote:
> I think you can use the entry node as the chain operand, rather than the store's chain?
Right, I was thinking of it as the chain for the split stores, but the copy from xzr can happen whenever.


https://reviews.llvm.org/D38146





More information about the llvm-commits mailing list