[PATCH] D107711: [DAGCombiner] Add target hook function to decide folding (mul (add x, c1), c2)

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

In D107711#2939527 <https://reviews.llvm.org/D107711#2939527>, @lebedev.ri wrote:

> Comments need updating





================
Comment at: llvm/include/llvm/CodeGen/TargetLowering.h:2086
+  /// The target should check the cost of materializing c1, c2 and c1*c2 into
+  /// registers. If it is not sure about some cases, a default true
+  /// can be returned to let the DAGCombiner decide.
----------------



================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.h:467
+  /// The target should check the cost of materializing c1, c2 and c1*c2 into
+  /// registers. If it is not sure about some cases, a default true
+  /// can be returned to let the DAGCombiner decide.
----------------



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

https://reviews.llvm.org/D107711