diff options
| author | Daniel Jasper <djasper@google.com> | 2014-02-20 11:10:48 +0000 |
|---|---|---|
| committer | Daniel Jasper <djasper@google.com> | 2014-02-20 11:10:48 +0000 |
| commit | 2f0f297bdb3f6f2127485b8b4dfbc9543b162139 (patch) | |
| tree | f96108648a959e2d5a99f52f962fe6bd32be98b6 /clang/lib/Basic/FileSystemStatCache.cpp | |
| parent | 1f2ce39006c583d0ab981bdb3d6ce7d64b1f365a (diff) | |
| download | llvm-2f0f297bdb3f6f2127485b8b4dfbc9543b162139.zip llvm-2f0f297bdb3f6f2127485b8b4dfbc9543b162139.tar.gz llvm-2f0f297bdb3f6f2127485b8b4dfbc9543b162139.tar.bz2 | |
Revert r201734 and r201742.
This breaks backwards compatibility with existing code. Previously, this
was defined as
#define _mm_prefetch(a, sel) (__builtin_prefetch((void *)(a), 0, (sel)))
Which basically accepts any pointer. Changing this to char* simply
breaks a lot of existing code. I have tried changing char* to
"const void*", which seems to be the right thing as per Intel
specification this should work on basically any pointer. However,
apparently this breaks windows compatibility (because of a conflicting
declaration in windows.h).
So, we probably need to #ifdef this based on whether clang is compiling
for windows. According to Chandler, this might be done by introducing an
additional symbol to a fake type in BuiltinsX86.def and then condition
the type expansion on the platform.
llvm-svn: 201775
Diffstat (limited to 'clang/lib/Basic/FileSystemStatCache.cpp')
0 files changed, 0 insertions, 0 deletions
