[PATCH] D124464: [AArch64] Replace `performANDSCombine` with `performFlagSettingCombine`.

Karl Meakin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 11 11:05:57 PDT 2022


Kmeakin marked 3 inline comments as done.
Kmeakin added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:18852-18853
     return performCSELCombine(N, DCI, DAG);
-  case AArch64ISD::ANDS:
-    return performANDSCombine(N, DCI);
   case AArch64ISD::DUP:
----------------
paulwalker-arm wrote:
> Is there a good reason to move this code?
To group it with other calls to `performFlagSettingCombine`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124464



More information about the llvm-commits mailing list