[PATCH] D134859: [clang][Interp] Implement basic support for floating point values

[Timm Bäder via Phabricator via cfe-commits]

tbaeder added inline comments.


================
Comment at: clang/lib/AST/Interp/PrimType.h:39
   PT_Bool,
+  PT_Float,
   PT_Ptr,
----------------
sepavloff wrote:
> In contrast to other members of this enumeration, `PT_Float` does not designate a concrete type, and it must creates problems. For example, how function `primSize(PrimType Type)` can be implemented?
Yes, floats are a bit special, see also the special case in `getCtorPrim()` in `Descriptor.cpp`. But I don't think it's a problem for `primSize()` since that only uses `sizeof()`.


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

https://reviews.llvm.org/D134859