[PATCH] Enhance urem to bitwise-and optimization

David Majnemer david.majnemer at gmail.com
Wed May 8 20:44:28 PDT 2013


On Wed, May 8, 2013 at 8:35 PM, Rafael EspĂ­ndola <rafael.espindola at gmail.com
> wrote:

> Is the select case really redundant now?

Yes, llvm::isKnownToBeAPowerOfTwo has support for select instructions. The
function 'test4' in rem.ll provides verification that we still handle
select instructions.


> Would you mind starting a FileCheck test file?
>
Should I start a FileCheck file or would it be better FileCheck-ize rem.ll?


>
> On 8 May 2013 18:22, David Majnemer <david.majnemer at gmail.com> wrote:
> > This patch replaces intelligence inside of InstCombiner::visitURem with a
> > call to isKnownToBeAPowerOfTwo.
> > By doing this, we can find more opportunities to transform urem
> instructions
> > to bitwise-and instructions.
> >
> > --
> > David Majnemer
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130508/011387b8/attachment.html>


More information about the llvm-commits mailing list