[PATCH] D78157: [AArch64InstrInfo] Ignore debug insts in areCFlagsAccessedBetweenInstrs [6/10]

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 16 13:23:33 PDT 2020


vsk marked 2 inline comments as done.
vsk added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:1188
+  --To;
+  for (const MachineInstr &Instr : reversedInstructionsWithoutDebug(To, From)) {
     if (((AccessToCheck & AK_Write) &&
----------------
fhahn wrote:
> nit: could be `reversedInstructionsWithoutDebug(std::prev(To), From)`.
Thanks, I'll fold this into the change before committing if there aren't any further comments.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78157





More information about the llvm-commits mailing list