[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
Thu Feb 22 06:09:04 PST 2018


asb added a comment.
Herald added a subscriber: shiva0217.

As @efriedma noted in https://reviews.llvm.org/D43106, it would be good to have some test coverage for ABI lowering. I'd suggest updating test/Driver/riscv32-abi.c with something like:

  #ifdef __SIZEOF_INT128__
  // CHECK-FORCEINT128-LABEL: define i128 @f_scalar_5(i128 %x)
  __int128_t f_scalar_5(__int128_t x) { return x; }
  #endif

You could also check the updated defines in in test/Preprocessor/init.c (seems it should just be the addition of `__SIZEOF_INT128__`).


Repository:
  rC Clang

https://reviews.llvm.org/D43105





More information about the cfe-commits mailing list