[PATCH] D44102: Teach CorrelatedValuePropagation to reduce the width of udiv/urem instructions.

Justin Lebar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 7 08:02:37 PST 2018


jlebar added a comment.

> To confirm - with this patch, we're getting all of the motivating cases (edge value propagation, local known bits, llvm.assume) that were affected by https://reviews.llvm.org/D37121? Ie, there's no current motivation to make instcombine try harder to shrink div/rem?

I still need to confirm, but I expect so.

I'm going to work on a patch to address the getPredicateAt problems Sanjoy pointed out above.

Also I need to revert this because it's crashing while building clang.  Will figure that out, it's probably something dumb.


Repository:
  rL LLVM

https://reviews.llvm.org/D44102





More information about the llvm-commits mailing list