[PATCH] D33850: Inlining: Don't re-map simplified cloned instructions.

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 30 16:37:58 PDT 2017


davide added a comment.

I looked at this more closely, and I personally don't see anything wrong with the code as-is.
We happen to have an instruction that simplifies to a different instruction in the original function (mainly because instsimplify does some hazy threading over select/phis), so maybe this assertion is too strict?
Kyle, what do you think?


Repository:
  rL LLVM

https://reviews.llvm.org/D33850





More information about the llvm-commits mailing list