tech-kern archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: Please do not yell at people for trying to help you.



On Fri, 12 Nov 2010, Joerg Sonnenberger wrote:

> On Fri, Nov 12, 2010 at 09:23:07PM +0000, Eduardo Horvath wrote:
> > On Sat, 13 Nov 2010, Valeriy E. Ushakov wrote:
> > 
> > > On Fri, Nov 12, 2010 at 21:59:54 +0100, Joerg Sonnenberger wrote:
> > > 
> > > > On Fri, Nov 12, 2010 at 08:31:39PM +0000, Eduardo Horvath wrote:
> > > > > The assignment:
> > > > > 
> > > > > foo.size = htole64(size);
> > > > > 
> > > > > Cannot be replaced with:
> > > > > 
> > > > > __inline __asm("stxa %1, [%0] ASI_LITLE" : &foo.size : size);
> > > > 
> > > > Actually, it should be possible to do exactly that if you allow the
> > > > output register to be a memory location. Not sure how smart GCC is for
> > > > this though.
> > > 
> > > Yeah, I don't see why memory operand contraint wouldn't work here.
> > > SPARC 'T' constraint - memory address aligned at 8 bytes.
> > 
> > 'Cause you're storing to foo.size which is not a parameter in the macro.
> 
> Doesn't really matter, as the compiler can reorder that and with the
> right constraints, it certainly should.

So instead of a single store you force a register spill, a load, and 
another store?  That's even worse than 8 shifts, ands and ors.

Eduardo


Home | Main Index | Thread Index | Old Index