[Lldb-commits] [lldb] [lldb][CompilerType] Add CompilerType::IsRealFloatingPointType (PR #178904)

[Michael Buch via lldb-commits]

================
@@ -2041,10 +2041,9 @@ static std::optional<clang::APValue> MakeAPValue(const clang::ASTContext &ast,
   if (is_integral)
     return clang::APValue(apint);
 
-  bool is_complex;
   // FIXME: we currently support a limited set of floating point types.
   // E.g., 16-bit floats are not supported.
-  if (!clang_type.IsFloatingPointType(is_complex))
+  if (!clang_type.IsRealFloatingPointType())
----------------
Michael137 wrote:

Yea this one isnt obvious but the Clang API that follows asserts if we're dealing with a non-real valued float. So i think we just never had coverage for that

https://github.com/llvm/llvm-project/pull/178904