[PATCH] D102834: [SLPVectorizer] Implement initial memory versioning.

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

In D102834#3029943 <https://reviews.llvm.org/D102834#3029943>, @fhahn wrote:

> @ABataev any suggestions on how to move forward with the patch?

I am not @ABataev :-) but I see you're defaulting `slp-memory-versioning` to false and was wondering about this. That looks like a safe strategy. But why is that exactly, as opposed to e.g. just enabling this by default? Is that because it's easier to test things before flipping the switch? How/when do you plan to flip this switch?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102834