[PATCH] D88419: [RDA] Switch isSafeToMove iterators

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 28 07:50:33 PDT 2020


samparker created this revision.
samparker added reviewers: SjoerdMeijer, samtebbs, nikic, dmgreen.
Herald added subscribers: hiraditya, kristof.beyls.
Herald added a project: LLVM.
samparker requested review of this revision.

So that forwards is forwards and backwards is reverse. Also add a check so that we know the instructions are in the expected order. We'll need a follow-up patch to two to be able to successfully move instructions around for ARMLowOverheadLoops.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D88419

Files:
  llvm/lib/CodeGen/ReachingDefAnalysis.cpp
  llvm/test/CodeGen/Thumb2/LowOverheadLoops/it-block-mov.mir
  llvm/test/CodeGen/Thumb2/LowOverheadLoops/lstp-insertion-position.mir
  llvm/test/CodeGen/Thumb2/LowOverheadLoops/mov-operand.ll
  llvm/test/CodeGen/Thumb2/LowOverheadLoops/move-def-before-start.mir
  llvm/test/CodeGen/Thumb2/LowOverheadLoops/move-start-after-def.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88419.294704.patch
Type: text/x-patch
Size: 11019 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200928/88f8bee8/attachment.bin>


More information about the llvm-commits mailing list