[PATCH] D111048: [ARM] Introduce a MQPRCopy

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp:1068
+      LLVM_DEBUG(dbgs() << " Unable to handle live out: " << *MI);
+      VMOVCopies.clear();
       return false;
----------------
samtebbs wrote:
> Does it need to clear the list if it's returning immediately after?
Yeah, It's a class variable used elsewhere. If we fail to tail predicate we might as well create vorr's, not vmovds for the code that remains.


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

https://reviews.llvm.org/D111048