This is an archive of the discontinued LLVM Phabricator instance.

[SVE] Remove VectorType::isScalable()
ClosedPublic

Authored by ctetreau on Apr 7 2020, 4:35 PM.

Details

Summary
  • This is a property of the instance of VectorType. If

isa<ScalableVectorType>(T) is true, then T->isScalable() would have
returned true and vice-versa. Most code that checks this function uses
the result to bail out if a vector is a scalable vector. This code will
be cleaner if it just calls isa<ScalableVectorType>(T)

Diff Detail

Event Timeline

ctetreau created this revision.Apr 7 2020, 4:35 PM
Herald added a project: Restricted Project. · View Herald Transcript
sdesmalen accepted this revision.Apr 22 2020, 8:43 AM

This interface seems unnecessary after adding ScalableVectorType, so LGTM!

This revision is now accepted and ready to land.Apr 22 2020, 8:43 AM
This revision was automatically updated to reflect the committed changes.