[PATCH] D43106: [RISCV] Enable -fuse-int128 through cmake flag COMPILER_RT_HAS_FINT128_FLAG

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 12 12:22:37 PST 2018


efriedma added inline comments.


================
Comment at: lib/builtins/CMakeLists.txt:505
   append_list_if(COMPILER_RT_HAS_STD_C11_FLAG -std=c11 BUILTIN_CFLAGS)
+  append_list_if(COMPILER_RT_HAS_FINT128_FLAG -fuse-int128 BUILTIN_CFLAGS)
 
----------------
kito-cheng wrote:
> kito-cheng wrote:
> > Does it possible only append this flag for target which has sizeof (long double) == 16 and !defined(__SIZEOF_INT128__)?
> Sorry I forgot to escape the code : `defined(__SIZEOF_FLOAT128__) && !defined(__SIZEOF_INT128__)`
I think it makes sense to always to force this, even if sizeof(long double) isn't 16; the 128-bit integer functions could be useful for other users.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D43106





More information about the llvm-commits mailing list