[PATCH] D97318: [clang][CodeGen] Allow fp16 arg pass by register

Yaxun Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 23 12:03:56 PST 2021


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



More information about the cfe-commits mailing list