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

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 12 08:49:30 PDT 2017


rengolin added a comment.

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

> I am wondering if we should have subtarget "owners" and then we could just file bugs (tasks) to enable such features on the "other" subtargets.  As I said I had a good results with SW data prefetching but for example the ARM microarchs didn't add support for this.


Hi Adam,

We have something similar for the release process (RELEASE_TESTERS.TXT), so it should be fine to have a list of people that volunteered to be bugged when sub-architectural decisions are needed.

If we go down that route, I suggest not to re-use CODE_OWNERS.TXT, because that's already a big mess. Maybe we could move all target owners from CODE_OWNERS to lib/Target/OWNERS.TXT and keep it hierarchical, so we can have multiple owners per target (as we already have in some).

Feel free to start this RFC on the mailing list, though, as this is not a discussion for this patch. :)

cheers,
--renato


https://reviews.llvm.org/D31965





More information about the llvm-commits mailing list