[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