Remove usages of asserting vector getters in Type in preparation for the
VectorType refactor. The existence of these functions complicates the
refactor while adding little value.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Event Timeline
| llvm/lib/IR/ConstantFold.cpp | ||
|---|---|---|
| 2010 | Please split out. | |
| llvm/lib/IR/Constants.cpp | ||
| 320 | Please split this off into a separate patch. I think "return false" is actually correct, but it probably makes sense to assert the vector is a ConstantAggregateZero or an UndefValue. | |
| llvm/lib/IR/Function.cpp | ||
| 1067 | Does VectorType::getNumElements() not return unsigned? | |
| llvm/lib/IR/Verifier.cpp | ||
| 4682 | Please don't mix functional and non-functional changes. | |
| llvm/lib/IR/Function.cpp | ||
|---|---|---|
| 1067 | it's uint64_t. getVectorNumElements() was unsigned, it just did the conversion | |
| llvm/lib/IR/Constants.cpp | ||
|---|---|---|
| 1894 | Please don't mix functional and non-functional changes. | |
| 1908 | Please don't mix functional and non-functional changes. | |
| llvm/lib/IR/Instructions.cpp | ||
| 2953 | Please don't mix functional and non-functional changes. | |
| 2972 | Please don't mix functional and non-functional changes. | |
| llvm/lib/IR/Verifier.cpp | ||
| 2830 | Please don't mix functional and non-functional changes. | |
| 3348 | Please don't mix functional and non-functional changes. | |
| 4663 | Please don't mix functional and non-functional changes. | |
| llvm/lib/IR/Constants.cpp | ||
|---|---|---|
| 231 | Please don't add new isScalable checks in this patch. (Here, and other places in this file.) | |
| llvm/lib/IR/ConstantFold.cpp | ||
|---|---|---|
| 757 | ConstantFoldSelectInstruction can take non-vector inputs. This should cause test failures. (Somehow I thought you fixed this in an earlier revision, but I could be mistaken.) | |
| llvm/lib/IR/ConstantFold.cpp | ||
|---|---|---|
| 757 | I'll fix it | |
| llvm/lib/IR/ConstantFold.cpp | ||
|---|---|---|
| 761 | Unnecessary cast<> | |
ConstantFoldSelectInstruction can take non-vector inputs. This should cause test failures. (Somehow I thought you fixed this in an earlier revision, but I could be mistaken.)