[PATCH] D97947: [AArch64] Force runtime unrolling for in-order scheduling models

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

This will still unroll the remainders of vectorized loops, which will be quite common but generally unhelpful to unroll. Can we try to prevent unrolling there too?



================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:1006
+  // Force runtime unrolling for in-order models
+  UP.Runtime |= !ST->getSchedModel().isOutOfOrder();
 }
----------------
Should we be enabling partial unrolling too? If not, why not?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D97947/new/

https://reviews.llvm.org/D97947