[PATCH] D108136: [LoopVectorize] Permit vectorisation of more select(cmp(), X, Y) reduction patterns

[David Sherwood via Phabricator via llvm-commits]

david-arm marked an inline comment as done.
david-arm added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/select-cmp.ll:290
+
+; We don't support the icmp and select using the same PHI.
+define i32 @select_i32_from_icmp_same_inputs(i32 %a, i32 %b, i64 %n) {
----------------
kmclaughlin wrote:
> Hi @david-arm, I'm not sure why we wouldn't support the icmp & select using the same PHI?
Well spotted @kmclaughlin! We can indeed support this.


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

https://reviews.llvm.org/D108136