Page MenuHomePhabricator

[VPlan] Handle scalarized values in VPTransformState.
Needs ReviewPublic

Authored by fhahn on Nov 29 2020, 1:32 PM.

Details

Reviewers
gilr
rengolin
Ayal
Summary

This patch adds plumbing to handle scalarized values directly in
VPTransformState.

Diff Detail

Event Timeline

fhahn created this revision.Nov 29 2020, 1:32 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 29 2020, 1:32 PM
fhahn requested review of this revision.Nov 29 2020, 1:32 PM
gilr added inline comments.Tue, Jan 19, 7:37 AM
llvm/lib/Transforms/Vectorize/VPlan.cpp
243

Why is this commented out?

llvm/lib/Transforms/Vectorize/VPlan.h
273

Better to use using?

fhahn updated this revision to Diff 318223.Thu, Jan 21, 8:48 AM

Address Gil's comments, thanks!

fhahn updated this revision to Diff 318225.Thu, Jan 21, 8:51 AM
fhahn marked an inline comment as done.

Remove unnecessary variable

llvm/lib/Transforms/Vectorize/VPlan.cpp
243

That was some leftovers from earlier experiments. I removed the line for now.