public inbox for libffi-discuss@sourceware.org
 help / color / mirror / Atom feed
From: Anthony Green <green@moxielogic.com>
To: Matthias Klose <doko@debian.org>
Cc: "libffi-discuss@sourceware.org" <libffi-discuss@sourceware.org>,
	"H.J. Lu" <hjl.tools@gmail.com>
Subject: Re: libffi trunk: new regressions on x86_64-linux-gnu, some test failures left on i686-linux-gnu
Date: Sat, 05 May 2018 13:52:00 -0000	[thread overview]
Message-ID: <CACxje59LJNox2njhcegGEKzMvphrRWn4DEc2Gd6LUkbMM7oNJw@mail.gmail.com> (raw)
In-Reply-To: <ee7a874b-d45e-baf4-0d0d-2a40ad1fa1f4@debian.org>

These failures are showing up because we're testing the microsoft ABI now.

This looks like a bug in GCC.  It's not returning small structures with
floating point values in the right registers when following the Microsoft
ABI.  I'm going to figure out how to 'xfail' those tests and open a bug
against GCC.   Clang appears to get it right.

AG


On Sat, May 5, 2018 at 9:39 AM, Matthias Klose <doko@debian.org> wrote:

> https://buildd.debian.org/status/package.php?p=libffi&suite=experimental
>
> trunk libffi testing from 2018-05-02, up to d3c54cf3a2b2bb2e889173b6a0a959
> 517b42c47f
>
> shows
>
>  - passing all tests on powerpc64-linux-gnu
>
>  - some regressions on x86_64-linux
>
> Running ../../testsuite/libffi.bhaible/bhaible.exp ...
> FAIL: libffi.bhaible/test-call.c -W -Wall -Wno-psabi -DDGTEST=55
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O0 -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-call.c -W -Wall -Wno-psabi -DDGTEST=55
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-call.c -W -Wall -Wno-psabi -DDGTEST=55
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -Os -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-call.c -W -Wall -Wno-psabi -DDGTEST=55
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -fomit-frame-pointer -DABI_NUM=FFI_WIN64
> -DABI_ATTR=__MSABI__ execution test
> FAIL: libffi.bhaible/test-call.c -W -Wall -Wno-psabi -DDGTEST=56
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O0 -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-call.c -W -Wall -Wno-psabi -DDGTEST=56
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-call.c -W -Wall -Wno-psabi -DDGTEST=56
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -Os -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-call.c -W -Wall -Wno-psabi -DDGTEST=56
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -fomit-frame-pointer -DABI_NUM=FFI_WIN64
> -DABI_ATTR=__MSABI__ execution test
> FAIL: libffi.bhaible/test-call.c -W -Wall -Wno-psabi -DDGTEST=57
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O0 -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-call.c -W -Wall -Wno-psabi -DDGTEST=57
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-call.c -W -Wall -Wno-psabi -DDGTEST=57
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -Os -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-call.c -W -Wall -Wno-psabi -DDGTEST=57
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -fomit-frame-pointer -DABI_NUM=FFI_WIN64
> -DABI_ATTR=__MSABI__ execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=54
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O0 -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=54
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=54
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -Os -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=54
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -fomit-frame-pointer -DABI_NUM=FFI_WIN64
> -DABI_ATTR=__MSABI__ execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=55
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O0 -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=55
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=55
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -Os -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=55
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -fomit-frame-pointer -DABI_NUM=FFI_WIN64
> -DABI_ATTR=__MSABI__ execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=56
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O0 -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=56
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=56
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -Os -DABI_NUM=FFI_WIN64 -DABI_ATTR=__MSABI__ execution
> test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=56
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -fomit-frame-pointer -DABI_NUM=FFI_WIN64
> -DABI_ATTR=__MSABI__ execution test
>
>
>  - improved results on i686-linux-gnu
>
> Running ../../testsuite/libffi.bhaible/bhaible.exp ...
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=46
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -Os -DABI_NUM=FFI_STDCALL -DABI_ATTR=__STDCALL__
> execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=48
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -Os -DABI_NUM=FFI_STDCALL -DABI_ATTR=__STDCALL__
> execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=49
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -Os -DABI_NUM=FFI_STDCALL -DABI_ATTR=__STDCALL__
> execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=51
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -Os -DABI_NUM=FFI_STDCALL -DABI_ATTR=__STDCALL__
> execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=53
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -DABI_NUM=FFI_STDCALL -DABI_ATTR=__STDCALL__
> execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=53
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -Os -DABI_NUM=FFI_STDCALL -DABI_ATTR=__STDCALL__
> execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=53
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -fomit-frame-pointer -DABI_NUM=FFI_STDCALL
> -DABI_ATTR=__STDCALL__ execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=60
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -DABI_NUM=FFI_STDCALL -DABI_ATTR=__STDCALL__
> execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=60
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -fomit-frame-pointer -DABI_NUM=FFI_STDCALL
> -DABI_ATTR=__STDCALL__ execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=61
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -Os -DABI_NUM=FFI_STDCALL -DABI_ATTR=__STDCALL__
> execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=64
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -DABI_NUM=FFI_STDCALL -DABI_ATTR=__STDCALL__
> execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=64
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -O2 -fomit-frame-pointer -DABI_NUM=FFI_STDCALL
> -DABI_ATTR=__STDCALL__ execution test
> FAIL: libffi.bhaible/test-callback.c -W -Wall -Wno-psabi -DDGTEST=65
> -Wno-unused-variable -Wno-unused-parameter -Wno-unused-but-set-variable
> -Wno-uninitialized -Os -DABI_NUM=FFI_STDCALL -DABI_ATTR=__STDCALL__
> execution test
>

  reply	other threads:[~2018-05-05 13:52 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-05 13:39 Matthias Klose
2018-05-05 13:52 ` Anthony Green [this message]
2018-05-05 14:36   ` Kaz Kylheku (libffi)
2018-05-06  1:31     ` Anthony Green
2018-05-06  1:31       ` Anthony Green
2018-05-05 13:52 ` H.J. Lu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CACxje59LJNox2njhcegGEKzMvphrRWn4DEc2Gd6LUkbMM7oNJw@mail.gmail.com \
    --to=green@moxielogic.com \
    --cc=doko@debian.org \
    --cc=hjl.tools@gmail.com \
    --cc=libffi-discuss@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).