[llvm] r274293 - Add LoadStoreVectorizer pass

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 1 11:44:03 PDT 2016


> On Jul 1, 2016, at 09:17, Filipe Cabecinhas via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> 
> Curious: Why is this a "GPU Load Store Vectorizer"?
> As a follow up, if there's no big reason, can you make the flag you
> committed on r274329 have a more generic name? This seems to be valid
> for any target.
> 
> Thank you,
> 
> Filipe
> 

It vectorizes just memory operations, unlike the SLPVectorizer which starts at stores and tries to vectorize the complete operation chain up to the loads.

It don’t think it should have a more generic name now since it isn’t added to the generic pass pipeline and is just in the backend. I’m not 100% sure this is the right pass order to run this and different targets probably want it in a different place.

-Matt


More information about the llvm-commits mailing list