[llvm-commits] [PATCH] Change Compiler.h to make ARM target works under MSVC

Howard Su howard0su at gmail.com
Thu May 21 05:30:15 PDT 2009


C++ standard (MSVC included) do support template specializations while it
doesn't support extern template specializations. So the following define is
wrong to me.
#ifdef __GNUC__
#define EXTERN_TEMPLATE_INSTANTIATION(X) __extension__ extern template X
#define TEMPLATE_INSTANTIATION(X) template X
#else
#define EXTERN_TEMPLATE_INSTANTIATION(X)
#define TEMPLATE_INSTANTIATION(X)   <======== wrong, C++ supports this.
#endif
Please verify.

-- 
-Howard
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20090521/6bf4adb2/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: build_under_vs8.patch
Type: application/octet-stream
Size: 604 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20090521/6bf4adb2/attachment.obj>


More information about the llvm-commits mailing list