[PATCH] D58435: [SCEV] Ensure that isHighCostExpansion takes into account what is being divided

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 1 12:20:53 PST 2019


RKSimon added inline comments.


================
Comment at: lib/Analysis/ScalarEvolutionExpander.cpp:2074
     // integer, consider the division cheap irrespective of whether it occurs in
     // the user code since it can be lowered into a right shift.
     if (auto *SC = dyn_cast<SCEVConstant>(UDivExpr->getRHS()))
----------------
Please update the comment


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

https://reviews.llvm.org/D58435





More information about the llvm-commits mailing list