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



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to