[llvm] Fix for logic in combineExtract() (PR #108208)

Ulrich Weigand via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 19 07:18:15 PDT 2024


uweigand wrote:

Hmm, I think the intent was for the type of the initial `Op` to be deliberately ignores, as it is supposed to be considered implicitly bitcast to `VecVT`.   But it is true that the routine assumes that `VecVT` must be a byte vector.  One of the two call sites (in combineTruncateExtract) ensures that, but the other call site (in combineEXTRACT_VECTOR_ELT) does not.  So I think the better fix would be to add a check to that call site instead.

https://github.com/llvm/llvm-project/pull/108208


More information about the llvm-commits mailing list