[PATCH] D121613: [LV] Use usesScalars in widenPHIInstruction.

[Ayal Zaks via Phabricator via llvm-commits]

Ayal accepted this revision.
Ayal added a comment.
This revision is now accepted and ready to land.

Ship it!

Ideally all other users of Cost->isScalarAfterVectorization() will also be refactored to query VPlan to retire this interface.



================
Comment at: llvm/test/Transforms/LoopVectorize/AArch64/sve-widen-gep.ll:147
+; CHECK-NEXT:    [[NEXT_GEP_0:%.*]] = getelementptr i8, i8* [[NEXT_GEP]], i32 0
+; CHECK-NEXT:    [[TMP16:%.*]] = bitcast i8* [[NEXT_GEP_0]] to <vscale x 2 x i8>*
 ; CHECK-NEXT:    [[WIDE_LOAD:%.*]] = load <vscale x 2 x i8>, <vscale x 2 x i8>* [[TMP16]], align 1
----------------
This is a case where we now know all users use scalars more accurately, where previously we generated vectors redundantly?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121613/new/

https://reviews.llvm.org/D121613