[PATCH] D41813: [builtins] Enable CRT_HAS_128BIT for MSVC

Loo Rong Jie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 8 04:56:58 PST 2018


rongjiecomputer updated this revision to Diff 128906.
rongjiecomputer added a comment.

@joerg You are right. I have checked that `__int128` is not available when compile with `-target i386-windows-msvc`.

Added extra guard with _WIN64.


https://reviews.llvm.org/D41813

Files:
  lib/builtins/int_types.h


Index: lib/builtins/int_types.h
===================================================================
--- lib/builtins/int_types.h
+++ lib/builtins/int_types.h
@@ -60,7 +60,8 @@
     }s;
 } udwords;
 
-#if (defined(__LP64__) || defined(__wasm__) || defined(__mips64))
+#if (defined(__LP64__) || defined(__wasm__) || defined(__mips64) || \
+     (defined(_MSC_VER) && defined(_WIN64)))
 #define CRT_HAS_128BIT
 #endif
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41813.128906.patch
Type: text/x-patch
Size: 417 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180108/71ead501/attachment.bin>


More information about the llvm-commits mailing list