--- Comment #21 from ubizjak at gmail dot com 2008-11-22 12:33 ---
This is a trace what happens in the testcase, from .expand dump:
(2) [frame + 8 ]- si
(3) [frame + 16]- dx
(4) r62 - di
(8) r63 - virtual-incoming-args + 0
(9) r64
--- Comment #22 from ubizjak at gmail dot com 2008-11-22 17:07 ---
Aliasing problems, gcc shoots himself in the foot...
When container consists of registers in different modes (due to
X86_64_INTEGERSI_CLASS optimization):
(parallel:BLK [
(expr_list:REG_DEP_TRUE (reg:DI 0 ax)
--- Comment #23 from howarth at nitro dot med dot uc dot edu 2008-11-22
17:39 ---
Patch in Comment 22 eliminates the va-arg test case failure at -m64 on
i686-apple-darwin9.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38151
--- Comment #24 from rguenth at gcc dot gnu dot org 2008-11-22 18:20
---
The va-arg code should probably use ref-all pointers all over the place
instead.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38151
--- Comment #25 from ubizjak at gmail dot com 2008-11-22 19:30 ---
Deassigning me, this is tree stuff.
--
ubizjak at gmail dot com changed:
What|Removed |Added
--- Comment #26 from rguenth at gcc dot gnu dot org 2008-11-22 20:41
---
Even with ref-all pointers we end up with
bb 5:
# addr.0{0}_1 = PHI va_arg_tmp.3(3), addr.0{0}_22(4)
# ap_38 = PHI ap_56(3), ap_57(4)
# va_arg_tmp.3_39 = PHI va_arg_tmp.3_55(3), va_arg_tmp.3_50(4)
--- Comment #27 from rguenth at gcc dot gnu dot org 2008-11-22 20:41
---
I have patches.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #15 from howarth at nitro dot med dot uc dot edu 2008-11-20
12:55 ---
The patch in comment 13 appears to be sufficient to completely fix the problem
on i686-apple-darwin9.
The results for current gcc trunk with the patch...
--- Comment #16 from ubizjak at gmail dot com 2008-11-20 19:50 ---
Problems from Comment #10 and Comment #11 are fixed by the patch from Comment
#13, but following test still fails, even with a patched compiler:
--cut here--
void abort (void);
struct S2848
{
unsigned int a;
--- Comment #17 from uros at gcc dot gnu dot org 2008-11-20 21:12 ---
Subject: Bug 38151
Author: uros
Date: Thu Nov 20 21:11:22 2008
New Revision: 142059
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=142059
Log:
PR target/38151
* config/i386/i386.c
--- Comment #18 from ubizjak at gmail dot com 2008-11-20 21:13 ---
va_arg problem from Comment #16 remains unfixed.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38151
--- Comment #19 from ubizjak at gmail dot com 2008-11-20 21:37 ---
Hm, rdx gets corrupted:
check2848va:
.LFB0:
.cfi_startproc
movq%rsi, %rcx # tmp73,
leaq8(%rsp), %rax #,
(+) movq%rdx, -40(%rsp) #,
shrq$32, %rcx #,
--- Comment #20 from howarth at nitro dot med dot uc dot edu 2008-11-21
00:05 ---
The test case in comment 16 passes on i686-apple-darwin9 when compiled with
-m32 but fails when compiled with -m64.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38151
--- Comment #12 from ubizjak at gmail dot com 2008-11-19 19:32 ---
Created an attachment (id=16723)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16723action=view)
Da patch.
Jack, can you try attached patch?
--
ubizjak at gmail dot com changed:
What|Removed
--- Comment #13 from ubizjak at gmail dot com 2008-11-19 21:17 ---
(In reply to comment #12)
Created an attachment (id=16723)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16723action=view) [edit]
Da patch.
Jack, can you try attached patch?
Patch at
--- Comment #14 from howarth at nitro dot med dot uc dot edu 2008-11-20
01:10 ---
The patch from Comment 13 when applied to gcc trunk (without a complete
bootstrap) eliminates the failures in gcc.dg-struct-layout-1 at -m64 while not
introducing any at -m32 on i686-apple-darwin9. I am
16 matches
Mail list logo
