[PATCH] D89549: [ARM][LowOverheadLoops] Check live-out for InsertPt instead of Start

Sam Tebbs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 19 03:27:58 PDT 2020


samtebbs added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp:1088
         InsertPt = FirstNonTerminator;
     } else if (RDA.hasSameReachingDef(Start, &*FirstNonTerminator, CountReg) &&
                RDA.hasSameReachingDef(Start, &*FirstNonTerminator, ARM::LR))
----------------
samparker wrote:
> Start -> InsertPt for these two as well.
I missed these, thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89549



More information about the llvm-commits mailing list