[llvm] Fixes inlining issue in armv7 (PR #169337)
David Green via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 1 03:55:39 PST 2025
================
@@ -64,37 +64,109 @@ class ARMTTIImpl final : public BasicTTIImplBase<ARMTTIImpl> {
const ARMTargetLowering *TLI;
// Currently the following features are excluded from InlineFeaturesAllowed.
- // ModeThumb, FeatureNoARM, ModeSoftFloat, FeatureFP64, FeatureD32
+ // ModeThumb, FeatureNoARM, ModeSoftFloat.
----------------
davemgreen wrote:
You added D32 but not F64. Should it be added too?
I would also add
- FeatureMatMulInt8, similar to DotProd
- HasMVEIntegerOps, HasMVEFloatOps, FeatureLOB, FeatureDFB are similar to other features already enabled.
- FeatureFuseLiterals,FeatureMVEVectorCostFactor1/2/4, FeatureNoPostRASched, FeaturePreferBranchAlign32/64, FeatureSplatVFPToNeon, FeatureUseMIPipeliner, FeatureUseMISched, FeatureUseWideStrideVFP, FeatureAvoidMULS are tuning features so should be safe.
https://github.com/llvm/llvm-project/pull/169337
More information about the llvm-commits
mailing list