[PATCH] D26217: Consider all SCEVMulExpr operands for factoring in FactorOutConstant

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 9 12:19:51 PST 2016


sanjoy added a comment.

In https://reviews.llvm.org/D26217#590831, @vhscampos wrote:

> Okay. Thanks for the clarification. Should I create another patch just for the removal of lines 223-225?


Sure.  Feel free to just check that patch in without pre-commit review.


https://reviews.llvm.org/D26217





More information about the llvm-commits mailing list