On 22/04/13 18:13, Richard Henderson wrote: > On 2013-04-22 09:23, Andreas Krebbel wrote: >> + /* We save registers 6-15. */ >> + long int __gregs[9]; > > Comment should be r6-r14, surely. > >> + /* r15 is stored into cfa field. It needs to be named that way >> + since tls.h is accessing the field by name. Be aware that this >> + is not actually what we consider the CFA on s390 (since there is >> + a fix offset between sp and CFA). It is merely the saved stack >> + pointer. */ >> + void *cfa; > ... >> + la %r3,96+96(%r15) /* second argument to GTM_begin_transaction */ > > Surely 96, not 96+96.
Fixed. Thanks! -Andreas-
