[LLVMbugs] Compile error on 64-bit x86 Solaris

Zeljko Vrba zvrba at ifi.uio.no
Wed Oct 24 10:01:59 PDT 2007


On Wed, Oct 24, 2007 at 10:21:54AM -0700, Chris Lattner wrote:
> 
> Two things:
> 1. Only .cpp files in lib/System should include <signal.h>.  If there are
>    others, we should eliminate them and move the system-specific
>    functionality there.
> 2. Assuming that you can't change solaris :), it would be fine to do this:
> 
> #include <signal.h>
> #undef CS  // hack for broken solaris headers
> #undef DS  // hack for broken solaris headers
> 

No, the problem is a "bit" more complicated: other LLVM files include
headers *other than* <signal.h>, and those headers in turn include
<signal.h>.  Grepping shows that <signal.h> is included only from the
following files:

./lib/System/Unix/Program.inc
./lib/System/Unix/Alarm.inc
./lib/System/Unix/Signals.inc

I tried to add #undefs in those files, but that did not fix the problem.




More information about the llvm-bugs mailing list