[PATCH] D37762: [InstCombine] Remove single use restriction from InstCombine's explicit sinking code.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

So I think we all agree that doing sinking in InstCombine isn't a good idea in the long term. But I suspect doing something to fix that is going to take some more work.

Do we think there any issues with this patch in the short term? Either significant performance regressions or compile time issues?


https://reviews.llvm.org/D37762