This is an archive of the discontinued LLVM Phabricator instance.

builtins: Fix the sysroot's reference to si_int in int_types.h
ClosedPublic

Authored by sgundapa on Nov 30 2015, 11:29 AM.

Details

Summary

si_int is already defined in sysroot's siginfo.h
emutls.c includes pthread.h which includes asm/siginfo.h which
in turn includes asm-generic/siginfo.h and that defines si_int.
si_int is also defined in builtin's int_types.h and this leads to
errors. This patch fixes the issue by undefining the si_int in int_types.h

Diff Detail

Repository
rL LLVM

Event Timeline

sgundapa updated this revision to Diff 41416.Nov 30 2015, 11:29 AM
sgundapa retitled this revision from to builtins: Fix the sysroot's reference to si_int in int_types.h.
sgundapa updated this object.
sgundapa added reviewers: joerg, rengolin.
sgundapa added a subscriber: llvm-commits.
joerg accepted this revision.Nov 30 2015, 11:32 AM
joerg edited edge metadata.

Looks good with the comment change.

lib/builtins/int_types.h
23 ↗(On Diff #41416)

Change that into Linux's asm-generic/siginfo.h please.

This revision is now accepted and ready to land.Nov 30 2015, 11:32 AM
This revision was automatically updated to reflect the committed changes.