[PATCH] D112300: [InstCombine] Don't split up Loads and free Exts

Mon Oct 25 07:12:59 PDT 2021

spatel added a comment.

In D112300#3084289 <https://reviews.llvm.org/D112300#3084289>, @avieira wrote:

> Fair enough, out of curiosity, where would you think this type of sinking should live in the optimization pipeline?

I don't have any intuition for the trade-offs, but in the past there were suggestions for a light-weight stand-alone pass that could be inserted as a clean-up at multiple points (similar to how we currently run -instcombine 5+ times in the typical pipeline).
Also, there have been proposals for stand-alone passes to do load/store transforms - D30416 <https://reviews.llvm.org/D30416> for example.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112300/new/

https://reviews.llvm.org/D112300