[llvm] 6594255 - [ARM] Add missing return to ARMTTIImpl::isLoweredToCall.

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 7 12:56:22 PDT 2022


Author: Craig Topper
Date: 2022-04-07T12:52:54-07:00
New Revision: 65942554e23ffcc5a48bb48d686b2f2986a59310

URL: https://github.com/llvm/llvm-project/commit/65942554e23ffcc5a48bb48d686b2f2986a59310
DIFF: https://github.com/llvm/llvm-project/commit/65942554e23ffcc5a48bb48d686b2f2986a59310.diff

LOG: [ARM] Add missing return to ARMTTIImpl::isLoweredToCall.

I assume we meant to return the result of the call to
BaseT::isLoweredToCall(F).

This might not be a functional change in practice because it would
still hit the default case in the switch and call
BaseT::isLoweredToCall(F) at the end.

Reviewed By: efriedma

Differential Revision: https://reviews.llvm.org/D123333

Added: 
    

Modified: 
    llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp b/llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
index 7e802238b362e..dcaec6606de17 100644
--- a/llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
+++ b/llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
@@ -1772,7 +1772,7 @@ ARMTTIImpl::getIntrinsicInstrCost(const IntrinsicCostAttributes &ICA,
 
 bool ARMTTIImpl::isLoweredToCall(const Function *F) {
   if (!F->isIntrinsic())
-    BaseT::isLoweredToCall(F);
+    return BaseT::isLoweredToCall(F);
 
   // Assume all Arm-specific intrinsics map to an instruction.
   if (F->getName().startswith("llvm.arm"))


        


More information about the llvm-commits mailing list