[haiku-bugs] Re: [Haiku] #5892: clang build fixes part 1 [patch]

  • From: "kaliber" <trac@xxxxxxxxxxxx>
  • Date: Mon, 03 May 2010 09:10:03 -0000

#5892: clang build fixes part 1 [patch]
-----------------------+----------------------------------------------------
 Reporter:  kaliber    |       Owner:  nobody        
     Type:  bug        |      Status:  new           
 Priority:  normal     |   Milestone:  R1            
Component:  - General  |     Version:  R1/Development
 Keywords:  clang      |   Blockedby:                
 Platform:  All        |    Blocking:                
-----------------------+----------------------------------------------------

Comment(by kaliber):

 Replying to [comment:1 stippi]:
 > The change to headers/posix/fenv.h is definitely not correct/complete,
 since clang != x86. Doesn't clang define _x86_? Looks like it should for
 compatibility reasons, so this should be fixed in the clang build for
 Haiku.

 It's looks like _X86_ is non standard define. Haiku uses _INTEL_ in many
 places, for example http://dev.haiku-
 os.org/browser/haiku/trunk/headers/os/kernel/OS.h#L407
 So we really should switch to _INTEL_ from _X86_.

-- 
Ticket URL: <http://dev.haiku-os.org/ticket/5892#comment:2>
Haiku <http://dev.haiku-os.org>
Haiku - the operating system.

Other related posts: