[PATCH] D37121: [DivRemHoist] add a pass to move div/rem pairs into the same block (PR31028)

Justin Lebar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 5 07:51:05 PST 2018


jlebar added a comment.

In https://reviews.llvm.org/D37121#1027143, @spatel wrote:

> Are you confident that the problem is limited to cases with a constant div/rem operand and masking of the variable that could be replaced by a trunc?


Actually I'm pretty confident the problem is *not* limited to such cases -- sorry for making a misleading testcase.  In the code I'm interested in, we know the bit-width of the dividend because of a branch on the dividend's value (x < some_constant) or because an llvm.assume tells us so.


Repository:
  rL LLVM

https://reviews.llvm.org/D37121





More information about the llvm-commits mailing list