lkml.org 
[lkml]   [2007]   [Sep]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch 1/2] Enable link power management for ata drivers
Davide Libenzi wrote:
> On Tue, 25 Sep 2007, roel wrote:
>
>>> + if (!(ap->flags & ATA_FLAG_IPM) || !ata_dev_enabled(dev)) {
>> if (!((ap->flags & ATA_FLAG_IPM) && ata_dev_enabled(dev))) {
>
> int foo(int i, int j) {
>
> return !(i & 8) || !j;
> }
>
> int moo(int i, int j) {
>
> return !((i & 8) && j);
> }
>
>
> gcc -O2 -S:
>
>
> .globl foo
> .type foo, @function
> foo:
> shrl $3, %edi
> xorl $1, %edi
> testl %esi, %esi
> sete %al
> orl %eax, %edi
> andl $1, %edi
> movl %edi, %eax
> ret
> .globl moo
> .type moo, @function
> moo:
> shrl $3, %edi
> xorl $1, %edi
> testl %esi, %esi
> sete %al
> orl %eax, %edi
> andl $1, %edi
> movl %edi, %eax
> ret

Indeed, no difference, except for the eye.

do you not consider it an improvement or do you not want to change it?
or
don't you consider it an improvement and want to change it?

Never mind. Disregard if you please.

> - Davide

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-09-25 02:03    [W:0.037 / U:0.140 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site