Edit detail for #48 #define unlikely defined in linux/compiler.h revision 1 of 1

changed:
-
I finally nailed its origin down to /usr/include/linux/compiler.h
In that file, we have
=====
/* Somewhere in the middle of the GCC 2.96 development cycle, we implemented
 a mechanism by which the user can annotate likely branch directions and
 expect the blocks to be reordered appropriately. Define __builtin_expect
 to nothing for earlier compilers. */
#if __GNUC__ == 2 && __GNUC_MINOR__ < 96
#define __builtin_expect(x, expected_value) (x)
#endif
#define likely(x) __builtin_expect((x),1)
#define unlikely(x) __builtin_expect((x),0)
=====
 
Notice that those defines are not guarded by __KERNEL__.
I would suggest that, Axiom spells its C MACROS all in CAPS, and
prefix them with AXIOM_.
Is that workable?
From unknown Mon Jan 24 18:42:59 -0600 2005
From: 
Date: 2005年1月24日 18:42:59 -0600
Subject: property change
Message-ID: <20050124184259-0600@page.axiom-developer.org>
Category: Axiom Compiler => building Axiom from source 

I finally nailed its origin down to /usr/include/linux/compiler.h
In that file, we have
=====
/* Somewhere in the middle of the GCC 2.96 development cycle, we implemented
 a mechanism by which the user can annotate likely branch directions and
 expect the blocks to be reordered appropriately. Define __builtin_expect
 to nothing for earlier compilers. */
#if __GNUC__ == 2 && __GNUC_MINOR__ < 96
#define __builtin_expect(x, expected_value) (x)
#endif
#define likely(x) __builtin_expect((x),1)
#define unlikely(x) __builtin_expect((x),0)
=====
 
Notice that those defines are not guarded by __KERNEL__.
I would suggest that, Axiom spells its C MACROS all in CAPS, and
prefix them with AXIOM_.
Is that workable?
------------------------------------------------------------
property change --2005年1月24日 18:42:59 -0600
Category: Axiom Compiler => building Axiom from source