[PATCH] D101103: [InstSimplify] Treat invariant group insts as bitcasts for load operands

[Reid Kleckner via Phabricator via llvm-commits]

rnk added a comment.

I added @lebedev.ri as a reviewer, since Roman has done significant work on instcombine/instsimplify.



================
Comment at: llvm/lib/Analysis/InstructionSimplify.cpp:5859
+
+  if (auto *BC = dyn_cast<BitCastOperator>(Op)) {
+    if (auto *C = ConstructLoadOperandConstant(BC->getOperand(0))) {
----------------
We already discarded constants above, so these don't have to be operators, they could be Instructions, BitCastInst, etc.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101103