[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