yaxunl marked an inline comment as done.
yaxunl added inline comments.
================
Comment at: clang/lib/CodeGen/TargetInfo.cpp:2825
+ // gcc where 16 bit integer is used in place of _Float16 or __fp16.
+ Lo = Integer;
}
----------------
tra wrote:
> Do we need to set `Hi`, too? We do set it for `int128`.
Seems not. By default Hi is NoClass. I tried setting Hi to Integer and it
resulted in an extra int64 argument of no use.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97318/new/
https://reviews.llvm.org/D97318
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
