[cfe-commits] Fix handling of ARM homogenous aggregates

Anton Korobeynikov anton at korobeynikov.info
Wed Apr 11 23:26:54 PDT 2012


Ping!

On Mon, Apr 9, 2012 at 18:03, Anton Korobeynikov
<anton at korobeynikov.info> wrote:
> Hi Bob et al
>
> Attached patch addresses the following issues:
>
>> * More seriously, I don't see how you can get away with asserting on unions in CodeGenFunction::ExpandTypeFromArgs.  Am I missing something?
>> * I don't much like the code duplication for the "expand a field" code in ExpandTypeToArgs.  If you can get away with just grabbing the first field of a union, I think you could unify that into a single loop over the fields and just exit after the first iteration in the case of a union.  Otherwise, maybe you could factor that code out into a separate function.
>
> I do not attach the testcases once again here just to simplify the patch :)
>
> --
> With best regards, Anton Korobeynikov
> Faculty of Mathematics and Mechanics, Saint Petersburg State University



-- 
With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University




More information about the cfe-commits mailing list