[PATCH] D135625: [instcombine] (extelt (inselt Vec, Value, Index), Index) -> Value

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 17 11:35:12 PDT 2022


dsanders added a comment.

> While the transform implemented here is correct, it is part of the wrong pass. This should be part of InstSimplify in simplifyExtractElementInst(). Can you please move it there?

Sure, I've done that in https://reviews.llvm.org/D136099


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135625



More information about the llvm-commits mailing list