[PATCH] headers: be a bit more careful about inline asm

Saleem Abdulrasool compnerd at compnerd.org
Tue Apr 29 22:12:29 PDT 2014


Hi rnk,

The Intrin.h header unconditionally includes ia32intrin.h.  It also
unconditionally defines inline assembly routines.  The assembly used here is x86
specific.  Be more careful about only providing definitions if we are targeting
x86.  This is now more relevant as clang can target Windows on ARM where these
declarations may be included into user code.

http://reviews.llvm.org/D3558

Files:
  lib/Headers/Intrin.h
  lib/Headers/ia32intrin.h
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3558.8955.patch
Type: text/x-patch
Size: 4637 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140430/c8a4cda6/attachment.bin>


More information about the cfe-commits mailing list