On Wed, Sep 5, 2012 at 12:49 AM, Steven Bosscher <stevenb....@gmail.com> wrote: > On Tue, Sep 4, 2012 at 6:43 PM, Andi Kleen <a...@firstfloor.org> wrote: >> +/* Compact representation of a index <-> resolution pair. Unpacked to an >> + vector later. */ >> +struct res_pair >> +{ >> + ld_plugin_symbol_resolution_t res; >> + unsigned index; >> +}; >> +typedef struct res_pair res_pair; >> + >> +DEF_VEC_P(res_pair); >> +DEF_VEC_ALLOC_P(res_pair, heap); > > Did you mean to use DEF_VEC_O here? > (Not sure it matters after the vec rewrite for c++)
Should be indeed _O. Ok with that change (double-check it still works with the recent VEC C++ changes). Ok to backport, check again if it works properly though! Thanks, Richard. > Ciao! > Steven