This is an archive of the discontinued LLVM Phabricator instance.

[VPlan] Use VPUser for VPWidenSelectRecipe operands (NFC).
ClosedPublic

Authored by fhahn on May 19 2020, 9:04 AM.

Details

Summary

VPWidenSelectRecipe already contains a VPUser, but it is not used. This
patch updates the code related to VPWidenSelectRecipe to use VPUser for
its operands.

Diff Detail

Event Timeline

fhahn created this revision.May 19 2020, 9:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 19 2020, 9:04 AM
gilr accepted this revision.May 24 2020, 12:33 AM
This revision is now accepted and ready to land.May 24 2020, 12:33 AM
Ayal added inline comments.May 24 2020, 1:31 AM
llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
4433

As noted earlier, would be good to avoid getting both ScalarCond and Cond, only to use one of them. Can be fixed separately.

This revision was automatically updated to reflect the committed changes.
fhahn marked an inline comment as done.May 24 2020, 9:20 AM
fhahn added inline comments.
llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
4433