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

kito-cheng added a comment. Herald added subscribers: VincentWu, luke957. From the psABI aspect, we already included that by this PR , this PR added size and alignment for `_Float16`, and we didn't added extra rule for that,

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

rogfer01 added a comment. I'm curious about how we handle `_Float16` here. My understanding from the psabi and the Zfh draft

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

craig.topper added a comment. In D112398#3087183 , @frasercrmck wrote: > It looks as though all checks are checking the same thing? Presumably this is > expected? I wonder if allowing an extra combined check >

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

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

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

HsiangKai created this revision. HsiangKai added reviewers: craig.topper, kito-cheng, frasercrmck, rogfer01. Herald added subscribers: achieveartificialintelligence, StephenFan, vkmr, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult,