Removing unnecessary instruction from ffi_call_unix64 (#588)
authorpetersn <schmidtnielsenpeter@gmail.com>
Tue, 27 Oct 2020 14:02:36 +0000 (07:02 -0700)
committerGitHub <noreply@github.com>
Tue, 27 Oct 2020 14:02:36 +0000 (10:02 -0400)
commitc78fcf88ab13a5cc414826a018dd46c3f886adee
tree044229f83512b4018c6356726d7695880d7ecf5d
parent2d86809db6d1d626a9df40c283ec564ac98ff85d
Removing unnecessary instruction from ffi_call_unix64 (#588)

unix64.S's `ffi_call_unix64` looks like it used to take six parameters,
where the sixth said the number of SSE register arguments. However,
currently the function only takes five parameters, and the number of SSE
register arguments is encoded in the `struct register_args *` passed as
the first parameter to `ffi_call_unix64`. This change removes an
instruction that tries to use this missing sixth parameter as the number
of SSE arguments.

This fix should not change any behavior, nor fix any bugs, because a few
instructions later the value moved from %r9d into %eax is overwritten by
the correct value anyway. This change merely makes the code a tad less
confusing, because currently the assembly moves from a register (r9)
whose value is never set.
src/x86/unix64.S