PATCH: Update sysdeps/x86_64/start.S for x32.

[H.J. Lu]

On Mon, May 14, 2012 at 4:04 PM, Roland McGrath <roland@hack.frob.com> wrote:
>> Is this how it shows up in diff.  I am also enclosing the patched file.
>
> Sorry, I thought I accounted for that when reading the diff but I guess I
> messed up.
>
>>       * sysdeps/x86_64/start.S: Include <sysdep.h>.  Simulate
>>       popping 4byte argument count for x32.  Use RXX_LP.
>
> The file already has #include <sysdep.h> so don't mention that.
>
> "4-byte"

Done.

> Say, "Use R*_LP macros."  RXX_LP looks like it could be the literal
> name of a macro rather than comment shorthand (and there are some
> instances where there is only one X).  Every instruction you touched
> to use R*_LP macros also had an operand-size suffix dropped, so in
> fact you should say, "Use R*_LP macros and omit operand-size suffix."

Done.

> At least one line you touched is >= 80 chars long.  It was before
> too, so even though you've made it longer you're not technically
> introducing the over-long line.  But fix it anyway while you're

Done and I fixed another long line while I was at it.

> touching it.  Sometimes you can just drop the second final space at
> the end of a whole-sentence comment to make it fit.  Sometimes you
> should just let M-; indent the comment differently to fit.
>
> You still need a space after a comma in the instructions added.

Done.

> Is it right that x32 uses full 8-byte stack slots for arguments
> that are of 4-byte types?
>

X32 kernel interface uses the same ia32 compat layer,
which uses 4-byte slot for passing the argument account
on stack from kernel to user space.

Here is the updated patch.  OK to install?

Thanks.


-- 
H.J.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0039-Simulate-popping-4byte-argument-count-for-x32.patch
Type: application/octet-stream
Size: 2458 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20120514/bbcf16d6/attachment.obj>