[PATCH] D86050: [SCEV] Refactor isHighCostExpansionHelper

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 23 22:41:01 PDT 2020


samparker added a comment.

> Please can you explain why the calculated costs changed for those X86 tests?

These were introduced once the Add and Mul expressions were costed in the same manner as when we're calculating for an AddRec. I'll remove the change for this patch and add it back in a follow-up.


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

https://reviews.llvm.org/D86050



More information about the llvm-commits mailing list