[PATCH] D93694: [MachineLICM][MachineSink] Move SinkIntoLoop from MachineLICM to MachineSink

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer updated this revision to Diff 317865.
SjoerdMeijer added a comment.

- Ported over the only positive test `test/CodeGen/X86/sink-cheap-instructions.ll` to a MIR test: this is `sink_add` in the new MIR test file.
- Walk candidates in reverse order, to start at the bottom of def-use chains, and try using the uses first before attempting to sink a def.


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

https://reviews.llvm.org/D93694

Files:
  llvm/lib/CodeGen/MachineLICM.cpp
  llvm/lib/CodeGen/MachineSink.cpp
  llvm/test/CodeGen/AArch64/loop-sink.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93694.317865.patch
Type: text/x-patch
Size: 49538 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210120/1f0702be/attachment.bin>