[PATCH] D137934: [InstCombine] Fold extractelt with select of constants

[Thomas Symalla via Phabricator via llvm-commits]

tsymalla added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/extractelement.ll:816
+}
+
 define i32 @extelt_select_const_operand_extractelt_use(i1 %c) {
----------------
arsenm wrote:
> Needs a test with an out of bounds vector index 
Hmm, I don't know if that makes sense. A trivial case with an `extractelement` with OOB behavior will not be caught by my code, as far as I can see, so it will be folded to a `poison` value.
I'll double-check and probably remove the OOB check in my code.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137934/new/

https://reviews.llvm.org/D137934