This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH v2, middle end]: Fix PR 37908, thinko with atomic NAND operation
- From: "Richard Guenther" <richard dot guenther at gmail dot com>
- To: "Andrew Haley" <aph at redhat dot com>
- Cc: "Jakub Jelinek" <jakub at redhat dot com>, "Uros Bizjak" <ubizjak at gmail dot com>, "David Daney" <ddaney at caviumnetworks dot com>, "Lu, Hongjiu" <hongjiu dot lu at intel dot com>, "Lavery, Daniel M" <daniel dot m dot lavery at intel dot com>, "GCC Patches" <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 30 Oct 2008 15:39:28 +0100
- Subject: Re: [PATCH v2, middle end]: Fix PR 37908, thinko with atomic NAND operation
- References: <4901BFC3.3090308@redhat.com> <8EA2C2C4116BF44AB370468FBF85A777A469033D@orsmsx504.amr.corp.intel.com> <49078759.7030709@caviumnetworks.com> <20081028220448.GF14706@tyan-ft48-01.lab.bos.redhat.com> <5787cf470810290105h4a7a900btf971a5259c278f86@mail.gmail.com> <49088A20.30308@caviumnetworks.com> <4908B47F.4030000@gmail.com> <4908BB64.5090206@redhat.com> <20081029194922.GM14706@tyan-ft48-01.lab.bos.redhat.com> <490980EB.3060000@redhat.com>
On Thu, Oct 30, 2008 at 10:39 AM, Andrew Haley <aph@redhat.com> wrote:
> Jakub Jelinek wrote:
>> On Wed, Oct 29, 2008 at 07:37:08PM +0000, Andrew Haley wrote:
>>> No, because real-world users might depend on the documented behaviour.
>>
>> Or maybe not. I've tried codesearch.google.com on both
>> __sync_fetch_and_nand and __sync_nand_and_fetch and it didn't reveal
>> anything outside of GCC (and LLVM) sources. I know that doesn't imply
>> no such users exist at all, but just that there probably aren't many
>> at least in the opensource world.
>
> Thanks for doing that. To be honest, the current behaviour isn't very
> useful, so I'm not surprised it isn't much used. I'm still going to
> suggest that silently changing the behaviour with no warning isn't a
> good idea, though.
If it is not used let us remove it.
Richard.
- References:
- Re: [PATCH, middle end]: Fix PR 37908, thinko with atomic NAND operation
- RE: [PATCH, middle end]: Fix PR 37908, thinko with atomic NAND operation
- Re: [PATCH, middle end]: Fix PR 37908, thinko with atomic NAND operation
- Re: [PATCH, middle end]: Fix PR 37908, thinko with atomic NAND operation
- Re: [PATCH v2, middle end]: Fix PR 37908, thinko with atomic NAND operation
- Re: [PATCH v2, middle end]: Fix PR 37908, thinko with atomic NAND operation
- Re: [PATCH v2, middle end]: Fix PR 37908, thinko with atomic NAND operation
- Re: [PATCH v2, middle end]: Fix PR 37908, thinko with atomic NAND operation
- Re: [PATCH v2, middle end]: Fix PR 37908, thinko with atomic NAND operation
- Re: [PATCH v2, middle end]: Fix PR 37908, thinko with atomic NAND operation