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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to