From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 3838A3971C67; Wed, 2 Sep 2020 14:16:08 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 3838A3971C67 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1599056168; bh=EtU90P3w5h7UW8HjcQR0gwlTDYxsE5vqAeLWvYS8SSA=; h=From:To:Subject:Date:In-Reply-To:References:From; b=P+5XrGXcwxJ8xuuyAQjrcB0JnberXBQ6geta+9u2gGmxXQMLdhyLTND8CyDdmjDmD Xacmn84DbrZ+LM2XbNdwRxcUUn7GNLVztMAiYsYoka+YCseAAxQknorAosmAOy2k2b GDixQ7fa/T7EpDX+PxKLs4XB0p8e/v86Hw3XpRHs= From: "rguenth at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug target/96895] ABI of returning V1DF differs between GCC and clang Date: Wed, 02 Sep 2020 14:16:08 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: target X-Bugzilla-Version: 11.0 X-Bugzilla-Keywords: ABI X-Bugzilla-Severity: normal X-Bugzilla-Who: rguenth at gcc dot gnu.org X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: gcc-bugs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-bugs mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 02 Sep 2020 14:16:08 -0000 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D96895 --- Comment #6 from Richard Biener --- (In reply to Richard Biener from comment #5) > (In reply to Michael Matz from comment #2) > > The psABI doesn't say anything about such types, no. Maybe it could in= some > > additional info pages, but it's always a problem to codify behaviour > > retroactively > > in it, when conflicting implementations already exist. It is about > > extension types, though, so we might be fine. > >=20 > > FWIW, even ignoring the obvious relation of v1Xf to Xf, GCC behaviour f= or > > float and clang behaviour for double is the most logical one: this exte= nded > > type is most > > similar to a struct containing one float/double, and such are passed in= XMM > > registers per psABI. As this is also consistent with how a single > > top-level float is passed, this choice is the most consistent one. Thi= s is > > also > > what the psABI _would_ say, if we had written it into it, so at least b= oth > > compilers would need a change to implement it. >=20 > So vector types with element type T and N, a power-of-two, not otherwise > specified are passes the same as >=20 > struct S { T a[N] }; >=20 > ? I guess there's mismatch then for AVX types with -mno-avx then at least > (AVX512 are probably too big to be passed in registers). Ugh. We pass struct S { double a[4]; } in %rdi _and_ on the stack?! And return by invisible reference _and_ in %eax? typedef struct S { double a[4]; } v4df; v4df foo (v4df x) { return x; } foo: .LFB0: .cfi_startproc movq %rdi, %rax movdqu 8(%rsp), %xmm0 movups %xmm0, (%rdi) movdqu 24(%rsp), %xmm1 movups %xmm1, 16(%rdi) ret but a two-element array is passed/returned in two %xmm regs=