[Mlir-commits] [mlir] [mlir][vector] Update the folder for vector.{insert|extract} (PR #136579)

Fehr Mathieu llvmlistbot at llvm.org
Tue Apr 22 07:00:28 PDT 2025


math-fehr wrote:

> Yes, good point — and actually @dcaballe has voiced support for treating any OOB access as poison (see https://github.com/llvm/llvm-project/issues/134516#issuecomment-2789687910):

Just mentionning this here so we don't forget, but if I recall @kuhar had a preference to immediate UB (https://github.com/llvm/llvm-project/issues/134516#issuecomment-2781890324).

And yes, I agree with you on all points, let's merge this first and then let's see the rest!

Also, just a quick question, why do we fold the `-1` instead of directly folding the operation to `ub.poison`?

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


More information about the Mlir-commits mailing list