[Lldb-commits] [PATCH] D16975: Handle floating-point type homogeneous aggregate return values in ABISysV_arm

[Tamas Berghammer via lldb-commits]

tberghammer added inline comments.

================
Comment at: source/Plugins/ABI/SysV-arm/ABISysV_arm.cpp:597
@@ +596,3 @@
+                {
+                    if (base_type.IsFloatingPointType(float_count, is_complex))
+                    {
----------------
omjavaid wrote:
> tberghammer wrote:
> > What is float_count means here? Do we have to check its value to decide if we can print the return value (e.g. what happens when float_count == 2)?
> float_count is just used to fullfil argument requirements. It was already declared in the function so didnt have to define it here.
> 
> It returns 1 if type is a standard builtin type (float or double), returns 2 for complex and number of elements for vector types.
> 
> We are using homogeneous_count to tell the number of elements in our aggregate type.
In that case I think we should check that its value after the call is 1 because we don't want to use the current implementation for vector types (we already check for complex types)


http://reviews.llvm.org/D16975