[s|z]exti ops do not have the same operand and result type.
As a consequence, the lowering of the n-D vector form needs to be relaxed a bit.
This revision additionally performs a few NFC renamings of variables to make them more intuitive.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
mlir/include/mlir/Conversion/StandardToLLVM/ConvertStandardToLLVM.h | ||
---|---|---|
658–659 | Can we just drop this? SameResultType does not seem to be used in the code, which seems to just support such ops now, | |
mlir/lib/Conversion/StandardToLLVM/StandardToLLVM.cpp | ||
1596 | If operandNDVector type is null, it would fail two lines above. Either change to dyn_cast there, or remove the check here. | |
mlir/test/Conversion/StandardToLLVM/convert-nd-vector-to-llvmir.mlir | ||
25–26 | Do we need this? |
Can we just drop this? SameResultType does not seem to be used in the code, which seems to just support such ops now,