[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