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

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added a comment.

In https://reviews.llvm.org/D37762#870140, @rriddle wrote:

> Has there been any sort of discussion on expanding/using the existing IR level code sinking pass? I am referring to the SinkingPass in scalar/sink.cpp. AFAIK it's only used in the AMDGPU preisel pipeline. I don't know it's current state/usability but the description of the pass is:
>
> // This pass moves instructions into successor blocks, when possible, so that
>  // they aren't executed on paths where their results aren't needed.


Doing this as a late IR pass makes sense. Taking a quick look at the implementation, there may be some improvements that would help (e.g., make it use MemorySSA, better handling of debug info, use a postdom tree). Considering adding this to the default codegen IR pipeline could be a good idea (and maybe better than trying to do a better job at the MI level?).


https://reviews.llvm.org/D37762