This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: Fix to PR41012

From: Xinliang David Li <davidxl at google dot com>
To: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Mon, 10 Aug 2009 10:14:04 -0700
Subject: Re: Fix to PR41012
References: <522e93240908082341j310891c6g9916b56a0c4dfd44@mail.gmail.com> <Pine.LNX.4.64.0908091412300.4585@digraph.polyomino.org.uk> <522e93240908091207l2f6c185epa43e89df04762f1a@mail.gmail.com>

Please see the attached file a new patch to fix the problem -- the
argmismatch  legality check is moved from lowering to the inliner
together with the rest of the checks.

Bootstrapped and regression tested on x86_64/linux.
SPEC2k int and SPEC06 testing with FDO.

Ok to checkin?

Thanks,

David

On Sun, Aug 9, 2009 at 12:07 PM, Xinliang David Li<davidxl@google.com> wrote:
> On Sun, Aug 9, 2009 at 7:13 AM, Joseph S. Myers<joseph@codesourcery.com> wrote:
>> On Sat, 8 Aug 2009, Xinliang David Li wrote:
>>
>>> The attached patch is meant to fix 41012. With this patch,
>>
>> Why does this patch not add a testcase to the testsuite?
>
> Will be added.
>
> Thanks,
>
> David
>>
>> --
>> Joseph S. Myers
>> joseph@codesourcery.com
>>
>

Attachment: inl_mismatch.p2
Description: Binary data

Attachment: inl.cg
Description: Binary data

Follow-Ups:
- Re: Fix to PR41012
  - From: Richard Guenther

References:
- Fix to PR41012
  - From: Xinliang David Li
- Re: Fix to PR41012
  - From: Joseph S. Myers

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]