[PATCH] D112398: [RISCV] Add ABI testing for Float16.

[Fraser Cormack via Phabricator via cfe-commits]

frasercrmck added a comment.

It looks as though all checks are checking the same thing? Presumably this is expected? I wonder if allowing an extra combined check (`--check-prefixes=CHECK,CHECK-ZFH-ILP32F` or something) would make it more obvious when things *are* different between the different configs.

I'm not familiar with how `Float16` is supposed to behave if the target doesn't advertise support `zfh`, but I come more from OpenCL where it's either fully supported or "storage-only", in which case I wouldn't expect a `fadd` to get past the frontend (or maybe it'd enforce promotion to `float`?). This isn't necessarily a blocker - I'm just showing the limits of my knowledge in this area.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112398