[PATCH] D36134: [ARM] Improve loop unrolling for Cortex-M

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 2 11:17:21 PDT 2017


efriedma added inline comments.


================
Comment at: lib/Target/ARM/ARMTargetTransformInfo.cpp:619
+    if (isa<SCEVCouldNotCompute>(Count) || !Count->getType()->isIntegerTy())
+      UP.DefaultUnrollRuntimeCount = 4;
+  }
----------------
This seems like you're running into some sort of limitation of unrolling infrastructure.  Maybe we need to add a feature to unroll remainder loops?

Also, which function in the test covers this codepath?


https://reviews.llvm.org/D36134





More information about the llvm-commits mailing list