[llvm] [ARM][FastISel]Fix FastISel fail for softfp when set target to armebv… (PR #97422)

[via llvm-commits]

https://github.com/hstk30-hw updated https://github.com/llvm/llvm-project/pull/97422

>From d8fa4817931601374490fab3500459c04d14e7f4 Mon Sep 17 00:00:00 2001
From: hstk <hanwei62 at huawei.com>
Date: Tue, 2 Jul 2024 22:09:55 +0800
Subject: [PATCH] [ARM][FastISel]Fix FastISel fail for softfp when set target
 to armebv7-unknown-linux

Related issue: https://github.com/llvm/llvm-project/pull/97422
---
 llvm/lib/Target/ARM/ARMFastISel.cpp     | 21 +++++++++++++++------
 llvm/test/CodeGen/ARM/fast-isel-call.ll | 11 +++++++++++
 2 files changed, 26 insertions(+), 6 deletions(-)

diff --git a/llvm/lib/Target/ARM/ARMFastISel.cpp b/llvm/lib/Target/ARM/ARMFastISel.cpp
index 61d2928fe6d412..b5b2c03a6e1622 100644
--- a/llvm/lib/Target/ARM/ARMFastISel.cpp
+++ b/llvm/lib/Target/ARM/ARMFastISel.cpp
@@ -1991,12 +1991,21 @@ bool ARMFastISel::ProcessCallArgs(SmallVectorImpl<Value*> &Args,
       assert(VA.isRegLoc() && NextVA.isRegLoc() &&
              "We only handle register args!");
 
-      AddOptionalDefs(BuildMI(*FuncInfo.MBB, FuncInfo.InsertPt, MIMD,
-                              TII.get(ARM::VMOVRRD), VA.getLocReg())
-                      .addReg(NextVA.getLocReg(), RegState::Define)
-                      .addReg(Arg));
-      RegArgs.push_back(VA.getLocReg());
-      RegArgs.push_back(NextVA.getLocReg());
+      if (MF->getDataLayout().isBigEndian()) {
+        AddOptionalDefs(BuildMI(*FuncInfo.MBB, FuncInfo.InsertPt, MIMD,
+                                TII.get(ARM::VMOVRRD), NextVA.getLocReg())
+                            .addReg(VA.getLocReg(), RegState::Define)
+                            .addReg(Arg));
+        RegArgs.push_back(NextVA.getLocReg());
+        RegArgs.push_back(VA.getLocReg());
+      } else {
+        AddOptionalDefs(BuildMI(*FuncInfo.MBB, FuncInfo.InsertPt, MIMD,
+                                TII.get(ARM::VMOVRRD), VA.getLocReg())
+                            .addReg(NextVA.getLocReg(), RegState::Define)
+                            .addReg(Arg));
+        RegArgs.push_back(VA.getLocReg());
+        RegArgs.push_back(NextVA.getLocReg());
+      }
     } else {
       assert(VA.isMemLoc());
       // Need to store on the stack.
diff --git a/llvm/test/CodeGen/ARM/fast-isel-call.ll b/llvm/test/CodeGen/ARM/fast-isel-call.ll
index eaf1850e620421..32bebc0ae4d45f 100644
--- a/llvm/test/CodeGen/ARM/fast-isel-call.ll
+++ b/llvm/test/CodeGen/ARM/fast-isel-call.ll
@@ -8,6 +8,8 @@
 ; RUN: llc < %s -O0 -verify-machineinstrs -fast-isel-abort=1 -relocation-model=dynamic-no-pic -mtriple=armv7-apple-ios -mattr=-fpregs | FileCheck %s --check-prefix=ARM-NOVFP
 ; RUN: llc < %s -O0 -verify-machineinstrs -fast-isel-abort=1 -relocation-model=dynamic-no-pic -mtriple=armv7-linux-gnueabi -mattr=-fpregs | FileCheck %s --check-prefix=ARM-NOVFP
 ; RUN: llc < %s -O0 -verify-machineinstrs -fast-isel-abort=1 -relocation-model=dynamic-no-pic -mtriple=thumbv7-apple-ios -mattr=-fpregs | FileCheck %s --check-prefix=THUMB-NOVFP
+; RUN: llc < %s -O0 -verify-machineinstrs -fast-isel-abort=1 -relocation-model=dynamic-no-pic -mtriple=armebv7-linux -mattr=+vfpv3 | FileCheck %s --check-prefix=ARMEB-SOFTFP
+; RUN: llc < %s -O0 -verify-machineinstrs -fast-isel-abort=1 -relocation-model=dynamic-no-pic -mtriple=armebv7 -mattr=+vfpv3 | FileCheck %s --check-prefix=ARMEB-SOFTFP
 
 ; Note that some of these tests assume that relocations are either
 ; movw/movt or constant pool loads. Different platforms will select
@@ -296,4 +298,13 @@ define void @call_undef_args() {
   ret void
 }
 
+define void @double_softfp_call() ssp {
+entry:
+; ARMEB-SOFTFP-LABEL: double_softfp_call:
+; ARMEB-SOFTFP: vmov r1, r0, {{d[0-9]+}}
+  call void @double_callee(double 0x3FC99999A0000000)
+  ret void
+}
+
 declare void @print(float)
+declare void @double_callee(double)