From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 31834 invoked by alias); 12 Feb 2012 22:15:55 -0000 Received: (qmail 31821 invoked by uid 22791); 12 Feb 2012 22:15:54 -0000 X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from ainaz.pair.com (HELO ainaz.pair.com) (209.68.2.66) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sun, 12 Feb 2012 22:15:40 +0000 Received: from ip-2-205-241-53.web.vodafone.de (ip-2-205-241-53.web.vodafone.de [2.205.241.53]) by ainaz.pair.com (Postfix) with ESMTPSA id 229B03F416; Sun, 12 Feb 2012 17:15:37 -0500 (EST) Date: Sun, 12 Feb 2012 22:23:00 -0000 From: Gerald Pfeifer To: Georg-Johann Lay cc: gcc-patches@gcc.gnu.org, Denis Chertykov , Eric Weddington Subject: Re: [Patch,wwwdocs,AVR]: AVR release notes In-Reply-To: <4F26A14D.4050207@gjlay.de> Message-ID: References: <4F19B2BB.2020401@gjlay.de> <4F26A14D.4050207@gjlay.de> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; CHARSET=US-ASCII X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2012-02/txt/msg00603.txt.bz2 On Mon, 30 Jan 2012, Georg-Johann Lay wrote: >> "Support for...has been added" (also typo: beed -> been) Hmm, this still seems to be in the latest version? > Is "C code" better? Or C-code? Without the extension, inline assembler > must be used to get correct code, using C like a = b or > pstruct->component will yield wrong code without the extensions if b > or *pstruct is located in flash. Thanks for the background. I really like how you have changed this in the new version of the patch! >> +
  • Support for AVR-specific built-in functions has beed added.
    • >> Which ones? > Must they all be named explicitly? Or is it ok to link to onlinedocs? > I'd prefer a link to the explanation in onlinedocs but I am unsure how > stable the links are as docs evolve over time/versions. If you think it's not beneficial, we don't have to do anything. The links should be rather stable in general (and I am running link checks somewhat regularly). +
  • Support has beed added for the built-in, 24-bit, signed and unsigned + integer types __int24 and __uint24.
    • I believe that should be "signed and unsigned 24-bit integer types" (omitting "built-in" should be fine, but in any case without commas). > What does "need no  " mean? Nothing at ",etc." all or blank ", etc."? I'd just use a regular space instead of   in that case. >> What is ".progmen"? Perhaps paraphrase this briefly? > Not easy without getting into too much technical details... Okay. > Attached an updated patch as there were many changes and so that Eric > and Denis can easier catch up. Looks good! Please fix the one typo, and consider the other comments as well, and go ahead and commit this I'd say. We can always make further changes later. This looks like an impressive release for AVR! Gerald