[PATCH] D111048: [ARM] Introduce a MQPRCopy

Sam Tebbs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 5 08:35:21 PDT 2021


samtebbs accepted this revision.
samtebbs added a comment.
This revision is now accepted and ready to land.

Looks good to me, with one more question.



================
Comment at: llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp:1068
+      LLVM_DEBUG(dbgs() << " Unable to handle live out: " << *MI);
+      VMOVCopies.clear();
       return false;
----------------
Does it need to clear the list if it's returning immediately after?


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

https://reviews.llvm.org/D111048



More information about the llvm-commits mailing list