[PATCH] D31965: [SLP] Enable 64-bit wide vectorization for Cyclone

Adam Nemet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 3 08:44:56 PDT 2017


anemet added a comment.

In https://reviews.llvm.org/D31965#744313, @kristof.beyls wrote:

> That sounds reasonable to me, but I would do it the other way around: enable it by default and explicitly disable it for the cores that we know have a chance of being evaluated and decided on later.
>  Otherwise, I'm afraid that we'll forever have an ever-growing whitelist of cores to enable this on, while it looks like the right thing to do in the end is to just enable it by default.


Sounds great, let me update the patch.


https://reviews.llvm.org/D31965





More information about the llvm-commits mailing list