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

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 3 08:39:31 PDT 2017


evandro 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.


Couldn't agree more with you, @kristof.beyls .


https://reviews.llvm.org/D31965





More information about the llvm-commits mailing list