Page MenuHomePhabricator

[NFC][LoopVectorizer] Remove VF.isScalable() assertion from collectInstsToScalarize and getInstructionCost
ClosedPublic

Authored by CarolineConcatto on Jul 20 2021, 6:38 AM.

Details

Summary

This patch removes the assertion when VF is scalable and replaces
getKnownMinValue() by getFixedValue(), so it still guards the code against
scalable vector types.
The assertions were used to guarantee that getknownMinValue were not used for
scalable vectors.

Diff Detail

Event Timeline

CarolineConcatto requested review of this revision.Jul 20 2021, 6:38 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 20 2021, 6:38 AM
sdesmalen accepted this revision.Jul 20 2021, 6:46 AM

LGTM. This is an NFC patch to simplify D101916.

This revision is now accepted and ready to land.Jul 20 2021, 6:46 AM
fhahn accepted this revision.Jul 20 2021, 7:09 AM
fhahn added a subscriber: fhahn.

LGTM, thanks!

This revision was landed with ongoing or failed builds.Jul 20 2021, 7:57 AM
This revision was automatically updated to reflect the committed changes.