[PATCH] D115394: [instcombine] Do demanded elts last when visiting extractelement

Thu Dec 9 06:29:12 PST 2021

spatel accepted this revision.
spatel added a comment.

LGTM

Side note on poison with FMF: this came up recently in D112117 <https://reviews.llvm.org/D112117>, and I'm not sure yet how to solve it. I think the issue is that "reassoc" effectively implies "nnan" + "ninf" + "nsz". Ie, if we are allowed to reorder FP math, anything is possible, and we can't be blamed for creating any of the special FP values...otherwise, we can't safely reorder any FP math.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115394