[PATCH] D62667: [ARM] Add the non-MVE instructions in Arm v8.1-M.

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 31 02:16:13 PDT 2019


dmgreen added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMScheduleA57.td:98
+
+  let UnsupportedFeatures = [HasV8_1MMainline, HasMVEInt, HasMVEFloat,
+                             HasFPRegsV8_1M];
----------------
samparker wrote:
> Why is this needed? And why just the A57?
I think this one is because the schedule has CompleteModel = 1 (it may be the only Arm schedule like that). A similar thing was done for SVE on older CPU's in the AArch64 backend not long ago.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62667





More information about the llvm-commits mailing list