[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