[llvm-commits] [PATCH] Emit andl instructions instead of andq instructions if possible

[Manman Ren]

On Dec 14, 2012, at 10:45 AM, David Majnemer wrote:

> Ping.
> 
> 
> On Sun, Dec 9, 2012 at 1:06 AM, David Majnemer <david.majnemer at gmail.com> wrote:
> There are cases where LLVM chooses to emit andq instructions where an andl instruction would suffice. The latter instruction is advantageous because it is a single byte smaller.
> 
> LLVM already had entries in X86InstrCompiler.td to perform this optimization, however another pattern was being preferred. The attached patch increases the priorities of two patterns so that they will be preferred is they match.

Hi David,

Can you please list the other patterns that were preferred before your patch? Just want to make sure it is okay to increase the priorities.

Thanks,
Manman

> 
> The attached patch passes all regressions and includes a test to verify that the fix is working.
> 
> _______________________________________________
> 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/20121214/9d9bcb34/attachment.html>