RFR: 8336938: Update libFFI to 3.4.6
Alexander Matveev
almatvee at openjdk.org
Tue Aug 13 01:05:51 UTC 2024
On Mon, 12 Aug 2024 23:15:58 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
> I see one minor difference between the upstream and your PR in a commented out line of code in `sysv_intel.S`. I don't know whether it is significant (I guess not, as long as it compiles with the current inline comment style).
>
> ```
> diff -u -w -r ../../jfx-tmp/jfx/rt/modules/javafx.media/src/main/native/gstreamer/3rd_party/libffi/src/x86/sysv_intel.S ./src/x86/sysv_intel.S
> --- ../../jfx-tmp/jfx/rt/modules/javafx.media/src/main/native/gstreamer/3rd_party/libffi/src/x86/sysv_intel.S 2024-08-12 15:59:14.597741100 -0700
> +++ ./src/x86/sysv_intel.S 2024-08-12 16:05:42.708534300 -0700
> @@ -102,7 +102,7 @@
> mov ecx, [12+ebp] /* load return type code */
> mov [ebp+8], ebx /* preserve %ebx */
> L(UW2):
> - // cfi_rel_offset(%ebx, 8)
> + /* cfi_rel_offset(%ebx, 8) */
>
> and ecx, X86_RET_TYPE_MASK
> lea ebx, [L(store_table) + ecx * 8]
> ```
Yes, it compiles fine. It was changed by libFFI itself in 3.4.6. I think to be inline with rest of comments in this file.
-------------
PR Comment: https://git.openjdk.org/jfx/pull/1531#issuecomment-2285153581
More information about the openjfx-dev
mailing list