[PATCH] D60691: [ARM] Replace fp-only-sp and d16 with fp64 and d32.

Tue May 28 03:19:56 PDT 2019

simon_tatham marked an inline comment as done.
simon_tatham added inline comments.

================
Comment at: llvm/test/MC/ARM/armv8.3a-js.s:16
 // REQ-V83: error: instruction requires: armv8.3a
-// REQ-FP: error: instruction requires: FPARMv8
+// REQ-FP: error: invalid instruction
----------------
ostannard wrote:
> Do you know why this diagnostic got worse?
I do now :-) It's because that instruction is invalid for two reasons: firstly, `vjcvt` requires FP-ARMv8 which the command line has turned off, and secondly, using d18 requires the new `d32` feature which the command line has also turned off. So `llc -debug` reports "Opcode result: multiple types of mismatch, so not reporting near-miss".

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60691