[libomptarget][nfc] Move non-freestanding headers out of common
Lowers the bar for building deviceRTL.
Drops math.h entirely as it wasn't used and libm is a big dependency.
Paths
| Differential D77071
[libomptarget][nfc] Move non-freestanding headers out of common ClosedPublic Authored by JonChesterfield on Mar 30 2020, 9:23 AM.
Details Summary [libomptarget][nfc] Move non-freestanding headers out of common Lowers the bar for building deviceRTL.
Diff Detail
Event TimelineJonChesterfield added inline comments.
JonChesterfield added inline comments.
This revision is now accepted and ready to land.Mar 30 2020, 7:38 PM Closed by commit rG09834f976125: [libomptarget][nfc] Move non-freestanding headers out of common (authored by JonChesterfield). · Explain WhyMar 31 2020, 4:00 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 254031 openmp/libomptarget/deviceRTLs/amdgcn/src/target_impl.h
openmp/libomptarget/deviceRTLs/common/debug.h
openmp/libomptarget/deviceRTLs/common/omptarget.h
openmp/libomptarget/deviceRTLs/common/src/data_sharing.cu
openmp/libomptarget/deviceRTLs/nvptx/src/target_impl.h
|
No stdio in amdgcn's target_impl as the only function used from it is printf, declared at the end of this header.
Can reduce amdgcn to wholly ffreestanding by providing an implementation of assert and two macros from inttypes, which may be a good idea as I think HIP is currently using the glibc versions of these headers for the device code.