[PATCH] D86530: [InstCombine] PHI-of-extractvalues -> extractvalue-of-PHI, aka invokes are bad

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In D86530#2238460 <https://reviews.llvm.org/D86530#2238460>, @lebedev.ri wrote:

> So before i look into `foldAggregateConstructionIntoAggregateReuse()` further,
> what are your thoughts about teaching InstSimplify to do most basic CSE for PHI nodes?
> (purely by comparing that the incoming values match)

Instruction::isIdenticalTo()? I see one use of that via visitStoreInst, so there's possible precedent. Start a llvm-dev thread to get other opinions?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86530