[PATCH] D62669: [ARM] Set up infrastructure for MVE vector instructions.

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


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

LGTM, just one minor nit.



================
Comment at: llvm/lib/Target/ARM/Disassembler/ARMDisassembler.cpp:126
+    private:
+      std::vector<unsigned char> VPTStates;
+  };
----------------
This could be a SmallVector<unsigned char, 4> to avoid some heap allocations.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62669





More information about the llvm-commits mailing list