Re: [PATCH] take 2 of the tr-based current

[Manfred Spraul]

This is a multi-part message in MIME format.
--------------000207020000080802050008
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit

Anton Blanchard wrote:

 >I changed the code a bit so that get_cpu() is now inline - this
 >represents our situation better. I think it is valid for gcc to cache
 >get_cpu across a function call in the below example because it knows
 >that get_cpu does not refer to any global variables.
 >
You are right: it seems that gcc internally decides if an inline
function is const or pure.

static inline int get_TR(void)
{
     int reg;
     __asm__ (
         "rep; nop;\n\t"
         "xor %0,%0\n\t"
         : "=r" (reg));
     return ret;
}

is always optimized as if get_TR is a function with __attribute__((const)).

Ben, is it intentional that get_TR is _not_ marked as inline? Your
version produces explicit function calls with -O2, and incorrect code
with -O99 (gcc decides to inline get_TR even without an inline
directive, and then optimizes away the calls after schedule.)

It seems that Anton's version generates the best code.
I've tested the attached version with egcs-1.1.2, gcc-2.96-98 and
gcc3-3.0.1-3, with -O0, -O2 and -O99.

--
     Manfred

--------------000207020000080802050008
Content-Type: application/msword;
 name="test.c"
Content-Transfer-Encoding: base64
Content-Disposition: inline;
 filename="test.c"

I2luY2x1ZGUgPHN0ZGlvLmg+CgpzdGF0aWMgdm9pZCBzY2hlZHVsZSh2b2lkKTsKCi8vLy8v
Ly8vIGN1dCBoZXJlIC8vLy8vLy8vLy8vLy8KCmV4dGVybiBpbnQgX19nZXRfVFJfd2FzX21p
c2NvbXBpbGVkOwpzdGF0aWMgaW5saW5lIHVuc2lnbmVkIGludCBnZXRfVFIodm9pZCkgX19h
dHRyaWJ1dGVfXyAoKHB1cmUpKTsKc3RhdGljIGlubGluZSB1bnNpZ25lZCBpbnQgZ2V0X1RS
KHZvaWQpCnsKCXVuc2lnbmVkIHRyOwoJX19hc21fXygic3RyICV3MCIKCQk6ICI9ZyIgKHRy
KQoJCTogIm0iIChfX2dldF9UUl93YXNfbWlzY29tcGlsZWQpKTsKCXJldHVybiB0cjsKfQov
Ly8vLy8vLyBjdXQgaGVyZSAvLy8vLy8vLy8vLwoKaW50IG1haW4odm9pZCkKewoJaW50IGNw
dTEsIGNwdTIsIGNwdTMsIGNwdTQ7CgljcHUxID0gZ2V0X1RSKCk7CgljcHUyID0gZ2V0X1RS
KCk7CglzY2hlZHVsZSgpOwoJY3B1MyA9IGdldF9UUigpOwoJY3B1NCA9IGdldF9UUigpOwoJ
cHJpbnRmKCJ0aGUgY3B1IHZhbHVlcyB3ZXJlICVkICVkICVkICVkLlxuIiwKCQljcHUxLCBj
cHUyLCBjcHUzLCBjcHU0KTsKCXJldHVybiAwOwp9CgpzdGF0aWMgaW50IGNwdTsKc3RhdGlj
IHZvaWQgc2NoZWR1bGUodm9pZCkKewoJY3B1ID0gMjsKCXByaW50Zigic2NoZWR1bGUgY2Fs
bGVkIC5cbiIpOwp9Cg==
--------------000207020000080802050008--