[llvm] [RISCV] Handle RVV return type in calling convention correctly (PR #87736)

Roger Ferrer Ibáñez via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 5 04:09:16 PDT 2024


================
@@ -1059,7 +1062,7 @@ class RVVArgDispatcher {
 
   unsigned CurIdx = 0;
 
-  void constructArgInfos(ArrayRef<Type *> TypeList);
+  template <typename Arg> void constructArgInfos(ArrayRef<Arg> Ret);
----------------
rofirrim wrote:

This is a matter of taste but I'd use two regular overloaded functions here rather than a template one  that you need to explicitly specialise later on the implementation file.

(I'm fine with the constructor of `RVVArgDispatcher` above because it is a definition.)

https://github.com/llvm/llvm-project/pull/87736


More information about the llvm-commits mailing list