[PATCH] D67375: [ARM] VFPv2 only supports 16 D registers.

Oliver Stannard (Linaro) via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 11 06:15:26 PDT 2019


ostannard added a comment.

This patch looks good, but presumably needs the clang half to be committed at the same time?

I considered suggesting using `vfp2d16` and `vfp2d16sp` instead of `vfp2` and `vfp2sp` to be consistent with the vfp3 feature names, but that would then be inconsistent with the user-facing name, so this way is fine.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D67375





More information about the llvm-commits mailing list