[PATCH] D72509: [ARM][LowOverheadLoops] Allow all MVE instrs.

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer accepted this revision.
SjoerdMeijer added a comment.
This revision is now accepted and ready to land.

LGTM, with just a nit inlined.



================
Comment at: llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp:479
 
-bool LowOverheadLoop::RecordVPTBlocks(MachineInstr* MI) {
+bool LowOverheadLoop::RecordMVEInsts(MachineInstr* MI) {
+  if (CannotTailPredicate)
----------------
nit: how about `RecordAllowedMVEInsts` to do a bit more justice what this function is doing?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72509