[PATCH] D79112: [SelectionDAG] Add the option of disabling generic combines.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 29 13:28:16 PDT 2020


arsenm added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:1655
+  SDValue RV;
+  if (!STI || !STI->disableGenericCombines(OptLevel))
+    RV = visit(N);
----------------
Why call this for every node? Why not just once when the combiner is constructed?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79112





More information about the llvm-commits mailing list