[PATCH] D44528: [PowerPC] Implement canCombineStoreAndExtract and provide the missing pattern for the combination

Lei Huang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 26 12:43:25 PDT 2018


lei added inline comments.


================
Comment at: test/CodeGen/PowerPC/combine-extract-store.ll:8
+; Function Attrs: norecurse nounwind
+define void @test(<4 x i32>* noalias nocapture readonly %VP, <4 x i32>* noalias nocapture %VP2, i32* noalias nocapture %IP) local_unnamed_addr #0 {
+; CHECK-LABEL: test:
----------------
nit: line past col 80



================
Comment at: test/CodeGen/PowerPC/combine-extract-store.ll:11
+; CHECK:       # %bb.0: # %entry
+; CHECK-NEXT:    vspltisw 2, 4
+; CHECK-NEXT:    lvx 3, 0, 3
----------------
Should these 2 lines be CHECK-DAG since they don't necessarily have to happen in this order?



Repository:
  rL LLVM

https://reviews.llvm.org/D44528





More information about the llvm-commits mailing list