[PATCH] D76135: [MachineLICM] Don't treat cross copies as cheap

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 13 11:18:01 PDT 2020


arsenm added a comment.

In D76135#1921963 <https://reviews.llvm.org/D76135#1921963>, @dmgreen wrote:

> Thanks for the suggestion.
>
> I made it call shouldRewriteCopySrc, which allows the target to modify things if it wishes. The AMDGPU/idiv-licm.ll test changed ever so slightly. Let me know if I should change things around to instead call shareSameRegisterFile more directly.


If DetectDeadLanes also has the same code, then they should probably be merged and directly exposed


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

https://reviews.llvm.org/D76135





More information about the llvm-commits mailing list