[PATCH] D111530: [TargetLowering] Optimize expanded SRL/SHL fed into SETCC ne/eq 0

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 7 05:16:48 PST 2022


RKSimon added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp:3572
+  Scan = [&Scan, &MatchShifts, &Result, OpSizeInBits](const SDValue &Value,
+                                                      unsigned Depth) {
+    if (Depth >= SelectionDAG::MaxRecursionDepth || !Value->hasOneUse())
----------------
You might be better off pulling this lambda out as static helper function instead of relying on recursive lambda calls?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111530



More information about the llvm-commits mailing list