[PATCH] D63247: [ARM] MVE VPT Block Pass

Oliver Stannard (Linaro) via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 13 05:34:14 PDT 2019


ostannard added a comment.

D62669 <https://reviews.llvm.org/D62669> includes a few VPT predicable MVE instructions to test the assembly side of this, is that enough to add some MIR tests for this?



================
Comment at: llvm/lib/Target/ARM/Thumb2ITBlockPass.cpp:395
+  bool Modified = false;
+  for (MachineFunction::iterator MFI = Fn.begin(), E = Fn.end(); MFI != E;) {
+    MachineBasicBlock &MBB = *MFI;
----------------
Can this one be a range-base for loop?


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

https://reviews.llvm.org/D63247





More information about the llvm-commits mailing list