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

Matthew Simpson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 3 04:37:14 PDT 2017


mssimpso added a comment.

In https://reviews.llvm.org/D31965#743954, @anemet wrote:

> There is no measurable compile-time change for AArch64 (testsuite, ctmark, spec).


Awesome!

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.


I'm fine with that plan as well.


https://reviews.llvm.org/D31965





More information about the llvm-commits mailing list