[PATCH] D86072: [SCEV] Cost Add and Mul Expr consistently

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 17 07:09:23 PDT 2020


samparker added a comment.

In D76434 <https://reviews.llvm.org/D76434> you highlighted by SCEVNAry expressions can have more than two operands, which would expand to a chain of operations, and the existing costs for AddRecExprs tries to account for. But this was missing for normal Add and Mul expressions. Have I misunderstood you?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86072



More information about the llvm-commits mailing list