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

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 8 15:36:31 PDT 2017


chandlerc added a comment.

In https://reviews.llvm.org/D33850#776482, @iteratee wrote:

> No longer needed because of https://reviews.llvm.org/D34017


I mean, it may not do *bad* things any more, but it still seems like an improvement? We shouldn't be doing the thing we're doing. Moving to an assert seems better?


Repository:
  rL LLVM

https://reviews.llvm.org/D33850





More information about the llvm-commits mailing list