[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