> >  // See gtm_thread::begin_transaction.
> > -uint32_t GTM::htm_fastpath = 0;
> > +extern "C"
> > +{
> > +  uint32_t __gtm_htm_fastpath = 0;
> 
> Please keep this as-is and rather use the same approach as for
> gtm_thread::begin_transaction.

What approach? 

I don't want to hardcode C++ mangling in the assembler file,
that just's horrible.

> 
> > +  uint32_t *__gtm_global_lock;
> 
> Same, and rearrange the serial lock's fields (see below).

My understanding is that C++ makes no guarantees of class layout.
That is why i ended up not relying on the layout.

-Andi

-- 
a...@linux.intel.com -- Speaking for myself only.

Reply via email to