[PATCH] D59357: [RISCV] Add codegen support for ilp32f, ilp32d, lp64f, and lp64d ("hard float") ABIs

[Alex Bradbury via Phabricator via llvm-commits]

asb updated this revision to Diff 192978.
asb marked an inline comment as done.
asb added a comment.

@rogfer01: Thanks for the review. You're right that we could do with coverage for varargs under the hard float ABIs. I've modifed test/CodeGen/RISCV/vararg.ll so it provides common coverage for vararg passing for these ABIs (which of course should be the same as the integer ABIs). How does this patch look now?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59357/new/

https://reviews.llvm.org/D59357

Files:
  lib/Target/RISCV/RISCVCallingConv.td
  lib/Target/RISCV/RISCVISelLowering.cpp
  lib/Target/RISCV/RISCVRegisterInfo.cpp
  test/CodeGen/RISCV/callee-saved-fpr32s.ll
  test/CodeGen/RISCV/callee-saved-fpr64s.ll
  test/CodeGen/RISCV/callee-saved-gprs.ll
  test/CodeGen/RISCV/calling-conv-ilp32-ilp32f-common.ll
  test/CodeGen/RISCV/calling-conv-ilp32-ilp32f-ilp32d-common.ll
  test/CodeGen/RISCV/calling-conv-ilp32d.ll
  test/CodeGen/RISCV/calling-conv-ilp32f-ilp32d-common.ll
  test/CodeGen/RISCV/calling-conv-lp64-lp64f-common.ll
  test/CodeGen/RISCV/calling-conv-lp64-lp64f-lp64d-common.ll
  test/CodeGen/RISCV/target-abi-valid.ll
  test/CodeGen/RISCV/vararg.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D59357.192978.patch
Type: text/x-patch
Size: 50450 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190330/044a34fc/attachment-0001.bin>