[PATCH] D123779: [AArch64] Add `foldOverflowCheck` DAG combine

Karl Meakin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 15 08:44:28 PDT 2022


Kmeakin marked an inline comment as done.
Kmeakin added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:15427-15430
+static bool isCMP(SDValue Op) {
+  return Op.getOpcode() == AArch64ISD::SUBS &&
+         !Op.getNode()->hasAnyUseOfValue(0);
+}
----------------
paulwalker-arm wrote:
> What's the value provided here? because I only see a single use.
It is used in later patches


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123779



More information about the llvm-commits mailing list