[PATCH] D44561: [ARM] Add HasFloat16 to TargetInfo

Tim Northover via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 16 07:41:33 PDT 2018


t.p.northover added inline comments.


================
Comment at: include/clang/Basic/TargetInfo.h:365
 
+  /// \brief Determine whether _Float16 is supported on this target.
+  virtual bool hasFloat16Type() const { return HasFloat16; }
----------------
SjoerdMeijer wrote:
> t.p.northover wrote:
> > `_Float16` doesn't seem to be supported anywhere in Clang (`__fp16` is).
> > 
> > But we should probably clarify exactly what kind of support we mean here. This variable doesn't affect:
> > 
> >   * Whether __fp16 can be used at all in source.
> >   * Its ABI.
> > 
> > I'm actually slightly worried that when we document what it does affect it'll end up being an ARM implementation-detail.
> > 
> > 
> I've added _Float16 support in Clang commit r312794: "Add _Float16 as a C/C++ source language type" :-)
Oh wow, that was ages ago! I have no idea how I still had a repo that old hanging around. Sorry about that.


https://reviews.llvm.org/D44561





More information about the cfe-commits mailing list