[PATCH] D43105: [RISCV] Enable __int128_t and uint128_t through clang flag

Alex Bradbury via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Feb 17 06:07:35 PST 2018


asb added a comment.

In https://reviews.llvm.org/D43105#1003709, @efriedma wrote:

> So you want int128_t for compiler-rt itself, so you can use the soft-float implementation, but you want to make int128_t opt-in to avoid the possibility of someone getting a link error trying to link code built with clang against libgcc.a?  That seems a little convoluted, but I guess it's okay.


That's one particular problem you might encounter if Clang and gcc for RISCV32 made different choices about enabling int128 by default. More generally, we want to ensure the C environment for both GCC and Clang is as close as possible, as any difference like this has the potential for causing difficult to debug differences in behaviour that confuse end users.


Repository:
  rC Clang

https://reviews.llvm.org/D43105





More information about the cfe-commits mailing list