[PATCH] D70009: [ARM][ReachingDefAnalysis] Use RDA for loloops

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp:358
+  if (SetFlags) {
+    if (RDA->getReachingDef(&MBB->back(), ARM::CPSR) > RDA->getInstId(MI))
+      SetFlags = false;
----------------
samparker wrote:
> dmgreen wrote:
> > Does this change need the Defs = [CPSR] for t2LoopDec change? Are there some tests for these cases that didn't change?
> It didn't need it in the end as this logic allows us to query for a CPSR def inbetween LoopDec (MI) and LoopEnd (back). Again, I'll create a helper which allows us to clearly whether we have a def of something inbetween A and B.
And we don't need to check for uses any more?


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

https://reviews.llvm.org/D70009