[PATCH] D81766: [VectorCombine] try to create vector loads from bitcasted scalar pointers

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In D81766#2091526 <https://reviews.llvm.org/D81766#2091526>, @nikic wrote:

> To add to what @lebedev.ri said, this patch violates the opaque pointer model towards which LLVM is migrating. Pointer element types are not allowed to influence optimization behavior.


Ok, let me see if I can rework this using just the cost model. This patch started within InstCombine, so we didn't have access to costs and didn't want to do the transform too loosely. The bitcast was used as a proxy for "cost effective" - it indicated that either the original code or the vectorizers had validated the vector type as a legitimate type for the target.


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

https://reviews.llvm.org/D81766