[PATCH] D147373: [InstCombine] fold double reverses

[Noah Goldstein via Phabricator via llvm-commits]

goldstein.w.n added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineVectorOps.cpp:2770
+  // TODO: (reverse (unaryop (reverse x))) -> (unaryop x)
+  // TODO: (reverse (binop (reverse x), y)) -> (binop x, (reverse y))
+  return nullptr;
----------------
Is there any different logic you need for binop that doesn't apply for cmp?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147373