[PATCH] D107082: [X86][RFC] Enable `_Float16` type support on X86 following the psABI

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

I haven't had a chance to look at this patch in detail, but I wanted to ask if you considered doing what ARM and RISCV do for this. They pass the f16 in the lower bits on an f32 by only changing the ABI handling code in the backend. The type legalizer takes care of the rest. That eems simpler than this patch. See for example https://reviews.llvm.org/D98670


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107082