[PATCH 00/20] Create and use hidden aliases in libgmp.so

Richard Henderson rth at twiddle.net
Mon Mar 4 21:58:32 CET 2013


On 03/04/2013 12:21 PM, Torbjorn Granlund wrote:
>   I've tried to do this in a series of steps that are as mechanical as
>   possible, and therefore as easy to review as possible.
>   
> Is the patch set intended to be applied as a whole, or will applying
> each (in number order) give something which is intended to work.

The patch set is intended to be fully bisectable.  Each patch in sequence
should work.  That said, I can't think of any real value in keeping the patches
separate in the final commit.  The intermediate steps are not useful.

> I don't see any problem which needs addressing.  I am perfectly happy
> with how debugging works with the asm code now.

Matter of taste, I suppose.

> You can do many things in m4 which I don't know how to handle in cpp.
> 
> E.g., for x86 and PIC, we need to remember a list of thunks to output at
> the end.

I generally do that with the assembler itself.  E.g.

	call	1f
	.subsection 2
1:	mov	(%esp), %ebx
	ret
	.previous
	addl	$_GLOBAL_OFFSET_POINTER_, %ebx

or whatever.


r~


More information about the gmp-devel mailing list