[PATCH] D89788: [RISCV] Add GHC calling convention

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 24 08:34:09 PDT 2020


jrtc27 added a comment.

What happens if you try and use a soft-float or single-float ABI with ghccc? Will GHC know to use integer registers instead, or will this cause errors that mean we need to validate the ABI+CC combination?



================
Comment at: llvm/test/CodeGen/RISCV/ghc-cc-rv32.ll:1
+; RUN: llc -mtriple=riscv32 -mattr=+f,+d --target-abi ilp32d < %s \
+; RUN:   | FileCheck %s
----------------
jrtc27 wrote:
> Please use update_llc_test_checks.py for these.
Please use "ghccc" rather than "ghc-cc" in the names of these files to match the calling convention name (as is done with the fastcc tests).


================
Comment at: llvm/test/CodeGen/RISCV/ghc-cc-rv32.ll:1-2
+; RUN: llc -mtriple=riscv32 -mattr=+f,+d --target-abi ilp32d < %s \
+; RUN:   | FileCheck %s
+
----------------
Please use update_llc_test_checks.py for these.


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

https://reviews.llvm.org/D89788



More information about the llvm-commits mailing list