[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