luismarques added inline comments.
================ Comment at: clang/lib/Driver/ToolChains/Arch/RISCV.cpp:386 + else + return Triple.getArch() == llvm::Triple::riscv32 ? "ilp32" : "lp64"; } ---------------- luismarques wrote: > When I compile a bare metal GNU toolchain (using > <https://github.com/riscv/riscv-gnu-toolchain>, reports GCC 8.3.0) I seem to > get lp64d by default. Should we not match that behaviour? > > ``` > > $ cat test.c > float foo() { return 42.0; } > $ riscv64-unknown-elf-gcc -O2 -S test.c > $ cat test.s > (...) > foo: > lui a5,%hi(.LC0) > flw fa0,%lo(.LC0)(a5) > (...) > ``` To clarify, that's a toolchain configured with `--enable-multilib`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65634/new/ https://reviews.llvm.org/D65634 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
