[CHERIoT] Pass two-cap-sized return values and arguments in registers

Author: xdoardo

clang/lib/CodeGen/Targets/RISCV.cpp

@@ -429,9 +429,40 @@ ABIArgInfo RISCVABIInfo::classifyArgumentType(QualType Ty, bool IsFixed,
     return ABIArgInfo::getIgnore();
 
   bool IsSingleCapRecord = false;
-  if (auto *RT = Ty->getAs<RecordType>())
+  bool FitsInTwoRegs = false;
+  auto *TD = Ty->getAsTagDecl();
+  if (TD && dyn_cast<RecordDecl>(TD)) {
+    auto *RD = dyn_cast<RecordDecl>(TD);
     IsSingleCapRecord = Size == getTarget().getCHERICapabilityWidth() &&
-                        getContext().containsCapabilities(RT->getDecl());
+                        getContext().containsCapabilities(RD);
+    FitsInTwoRegs = IsSingleCapRecord;
+
+    // Check if the record fits in two registers, that is:
+    // 1. it has two fields
+    // 2. any of the two fields is either a capability or a type whose size can
+    // fit in the data part of a register
+    if (!IsSingleCapRecord) {
+      int SeenFields = 0;
+      for (const FieldDecl *Field : RD->fields()) {
+        if (++SeenFields > 2) {
+          FitsInTwoRegs = false;
+          break;
+        }
+
+        auto FieldTy = Field->getType();
+        auto FieldInfo = Field->getASTContext().getTypeInfo(FieldTy);
+        if (!FieldTy->isAnyPointerType() &&
+            FieldInfo.Width > getTarget().getRegisterWidth()) {
+          FitsInTwoRegs = false;
+          break;
+        }
+
+        if (SeenFields == 2) {
+          FitsInTwoRegs = true;
+        }
+      }
+    }
+  }
 
   bool IsCapability = Ty->isCHERICapabilityType(getContext()) ||
                       IsSingleCapRecord;
@@ -528,7 +559,7 @@ ABIArgInfo RISCVABIInfo::classifyArgumentType(QualType Ty, bool IsFixed,
     return ABIArgInfo::getDirect();
   }
 
-  if (IsSingleCapRecord)
+  if (FitsInTwoRegs)
     return ABIArgInfo::getDirect();
 
   if (const VectorType *VT = Ty->getAs<VectorType>())