[PATCH] D106164: [LV] Don't assume isScalarAfterVectorization if one of the uses needs widening.

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen created this revision.
Herald added a subscriber: hiraditya.
sdesmalen requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

This fixes an issue that was found in D105199 <https://reviews.llvm.org/D105199>, where a GEP instruction
is used both as the address of a store, as well as the value of a store.
For the former, the value is scalar after vectorization, but the latter
(as value) requires widening.

Other code in that function seems to prevent similar cases from happening,
but it seems this case was missed.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D106164

Files:
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Transforms/LoopVectorize/AArch64/sve-widen-gep.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D106164.359377.patch
Type: text/x-patch
Size: 5646 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210716/20c3b723/attachment.bin>