Page MenuHomePhabricator

[PATCH 10/27] [noalias] Loop vectorizer: learn about noalias intrinsics
Needs ReviewPublic

Authored by jeroen.dobbelaere on Oct 4 2019, 2:44 PM.

Details

Reviewers
hfinkel
jdoerfert
Summary

This is part of the series started by D68484.

Note: this is a stable point and tests should run fine with the patches applied up to this point.
Note: D68498 has been incorporated.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptOct 4 2019, 2:44 PM
lebedev.ri added inline comments.
llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
3223–3227

Why was this moved? Just as an optimization in case we early-return inbetween?

jeroen.dobbelaere marked 2 inline comments as done.Oct 7 2019, 4:07 AM
jeroen.dobbelaere added inline comments.
llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
3223–3227

By moving this later, we avoid trying to get the vector variant of MetadataAsValue for the noalias intrinsics.
(and also the corresponding assertion)

jeroen.dobbelaere retitled this revision from [PATCH 16/38] [noalias] Loop vectorizer: learn about noalias intrinsics to [PATCH 09/26] [noalias] Loop vectorizer: learn about noalias intrinsics.
jeroen.dobbelaere edited the summary of this revision. (Show Details)
jeroen.dobbelaere retitled this revision from [PATCH 09/26] [noalias] Loop vectorizer: learn about noalias intrinsics to [PATCH 10/27] [noalias] Loop vectorizer: learn about noalias intrinsics.

Rebased to 9fb46a452d4e5666828c95610ceac8dcd9e4ce16 (September 7, 2020)