[PATCH] [PATCH] clang internal library headers changes for C++ modules

John Thompson john.thompson.jtsoftware at gmail.com
Thu Apr 23 12:09:55 PDT 2015


Hi rsmith,

These changes help us fix some issues related to C++ modules, and will help us diverge less from trunk in our SDK headers.

Create new file _denormals.h to contain duplicate definitions from pmmintrin.h (and xmmintrin.h, actually our version only, which also needed the denormal definitions).
Change pmmintrin.h and xmmintrin.h to include _denormals.h and delete duplicate definitions.
Change mm_malloc.h to not include stdlib.h and size_t, as this creates a circular module reference. Locally declare malloc and free with a size_t equivalent type, and replace other size_t references.
Create new file _gnu_va_list.h to define __builtin_va_list.  We had duplicate definitions of this locally, so we broke it out into a common header.
Change stdarg.h to include _gnu_va_list.h and delete the definition of __builtin_va_list there.

http://reviews.llvm.org/D9229

Files:
  lib/Headers/_denormals.h
  lib/Headers/_gnuc_va_list.h
  lib/Headers/mm_malloc.h
  lib/Headers/module.modulemap
  lib/Headers/pmmintrin.h
  lib/Headers/stdarg.h
  lib/Headers/xmmintrin.h

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D9229.24320.patch
Type: text/x-patch
Size: 7436 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150423/dd70a895/attachment.bin>


More information about the cfe-commits mailing list