[cfe-commits] Fix handling of ARM homogenous aggregates

Anton Korobeynikov anton at korobeynikov.info
Fri Mar 30 12:06:40 PDT 2012

> It seems that I misunderstood the meaning of the "isAggregate" method.  I didn't realize that having private fields would cause that to return false.  As long as you exclude classes with vtables, that should be fine.
It's testing for C++ aggregate, so it should not contain user-defined
ctors (no problem for h. aggregates), no private or protected
non-static members (no problem for h. aggregates), no base classes (no
problems for h. aggregates) and not virtual functions.

So, the check isAggregate() is not correct... I will add testcase for
vtables as well.

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

More information about the cfe-commits mailing list