[PATCH] D125765: [RISCV] Add type aliases float16_t, float32_t and float64_t

[Yueh-Ting (eop) Chen via Phabricator via cfe-commits]

eopXD added a comment.

Sorry for the late reply, I missed this in may mails.

I see that the motivation start from where you spotted this type alias. However it was essentially just an act for convenience so we don't have to map a more irregular pattern of ( _Float16, float, double) when generating the test cases by code. Is it necessary that we have them in the header? On the other hand, these are scalar floating-point types, and if we want them to be defined, I agree with Zakk that starting a discussion in riscv-c-api will be helpful.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125765