[PATCH] AAPCS-VFP: double and long double have the same machine	type
    Renato Golin 
    renato.golin at linaro.org
       
    Thu Feb  6 14:13:45 PST 2014
    
    
  
On 6 February 2014 16:31, Oliver Stannard <oliver.stannard at arm.com> wrote:
> Having said that, it took long enough to convince myself that version two
> of the patch is correct that I have re-written it to make it clearer, new
> version of the patch is attached.
>
Hi Oliver,
It looks a lot better now, thanks! It was a bit of a mess and I got
completely confused, which would also have confused future developers, too.
:)
LGTM, apart from an unnecessary white-space change in an unrelated file:
-  "fits_in_regs");
+                          "fits_in_regs");
cheers!
--renato
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140206/fef6042d/attachment.html>
    
    
More information about the cfe-commits
mailing list