[SVE] Remove VectorType::getBitWidth()

[ctetreau]

Summary

• VectorType::getBitWidth() is just an unsafe version of

getPrimitiveSizeInBits() that assumes all vectors are fixed width.

[efriedma]

Makes sense.

llvm/lib/IR/Type.cpp

75

This seems a little messy. Maybe separate the vector/vector case from the vector/mmx case?

[ctetreau]

address code review issues

[efriedma]

LGTM

[ctetreau]

rebase

[ctetreau]

rebase

[ctetreau]

rebase

[ctetreau]

Make this commit not depend on the existence of FixedVectorType

[efriedma]

llvm/lib/IR/Type.cpp

77

I think you broke this in the process of making it not depend on FixedVectorType?

[ctetreau]

catch stragglers

[ctetreau]

llvm/lib/IR/Type.cpp

77

Yes I did, good catch. If only there were a FixedVectorType :)

[ctetreau]

fix dumdum

[efriedma]

LGTM. Please make a note to clean up Type::canLosslesslyBitCastTo later.

[ctetreau]

In D77833#1995390, @efriedma wrote:

LGTM. Please make a note to clean up Type::canLosslesslyBitCastTo later.

It's cleaned up in D77587 which I plan to merge right after this goes in