benshi001 added a comment. In D103878#2809033 <https://reviews.llvm.org/D103878#2809033>, @luismarques wrote:
> In D103878#2807118 <https://reviews.llvm.org/D103878#2807118>, @benshi001 > wrote: > >> 1. there is no tests for mabi=ilp32e, and my patch covers that. >> 2. the tests in riscv-abi.c will show default abi changes for special archs, >> especially for the arch with F but without D extension, in the future. >> 3. the tests in riscv-arch.c will show default arch changes for abi=ilp32, >> which is rv32imacfd now, and it is better to be rv32imac. for abi=ilp32f it >> is better arch=imacf than current imacfd. > > That sounds like a good description to add to the patch summary / commit > message! Done. Thanks. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103878/new/ https://reviews.llvm.org/D103878 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits