[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