[PATCH] D30732: LoopVectorizer: let target limit memory intensive loops

Matthew Simpson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 8 12:22:51 PST 2017


mssimpso added a comment.

I think it might be better if the store-counting part of this was located in the SystemZ TTI implementation. So the hook would look something like TTI.hookName(WideningDecions, VF). What do you think? Letting the targets see all the memory access decisions would probably be more generally useful. And I believe the only thing that would have to be exposed is the InstWidening enum.


https://reviews.llvm.org/D30732





More information about the llvm-commits mailing list